Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed test failures #105

Merged
merged 4 commits into from
May 21, 2024
Merged

Fixed test failures #105

merged 4 commits into from
May 21, 2024

Conversation

nkvuong
Copy link
Contributor

@nkvuong nkvuong commented May 16, 2024

Nightly is failing due to #103

  • Made ProductInfo.version a cached_property, to avoid failure when comparing wheel uploads in development
  • Sorted upgrade scripts to ensure they are applied in semver order

Copy link

codecov bot commented May 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.90%. Comparing base (2b75d24) to head (cfffae2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   78.78%   78.90%   +0.11%     
==========================================
  Files          14       14              
  Lines        1499     1498       -1     
  Branches      269      268       -1     
==========================================
+ Hits         1181     1182       +1     
+ Misses        230      229       -1     
+ Partials       88       87       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nkvuong nkvuong changed the title fix test failures due to #103 Fixed test failures May 21, 2024
Copy link
Contributor

@nfx nfx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@nfx nfx merged commit 2511cf5 into databrickslabs:main May 21, 2024
12 of 13 checks passed
nfx added a commit that referenced this pull request May 21, 2024
* Made `ProductInfo.version` a `cached_property` to avoid failure when comparing wheel uploads in development ([#105](#105)). In this release, the `apply` method of a class has been updated to sort upgrade scripts in semantic versioning order before applying them, addressing potential issues with version comparison during development. The implementation of `ProductInfo.version` has been refactored to a `cached_property` called `_version`, which calculates and caches the project version, addressing a failure during wheel upload comparisons in development. The `Wheels` class constructor has also been updated to include explicit keyword-only arguments, and a deprecation warning has been added. These changes aim to improve the reliability and predictability of the upgrade process and the library as a whole.

Dependency updates:

 * Bump actions/checkout from 4.1.5 to 4.1.6 ([#106](#106)).
@nfx nfx mentioned this pull request May 21, 2024
nfx added a commit that referenced this pull request May 21, 2024
* Made `ProductInfo.version` a `cached_property` to avoid failure when
comparing wheel uploads in development
([#105](#105)). In
this release, the `apply` method of a class has been updated to sort
upgrade scripts in semantic versioning order before applying them,
addressing potential issues with version comparison during development.
The implementation of `ProductInfo.version` has been refactored to a
`cached_property` called `_version`, which calculates and caches the
project version, addressing a failure during wheel upload comparisons in
development. The `Wheels` class constructor has also been updated to
include explicit keyword-only arguments, and a deprecation warning has
been added. These changes aim to improve the reliability and
predictability of the upgrade process and the library as a whole.

Dependency updates:

* Bump actions/checkout from 4.1.5 to 4.1.6
([#106](#106)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants