-
Notifications
You must be signed in to change notification settings - Fork 660
[DYN-9230] Bump dynamo major version check for PackageManager. #16428
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9230
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs
Outdated
Show resolved
Hide resolved
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR bumps the Dynamo major version check from 3 to 4 in package manager compatibility logic to align with the current Dynamo version. This change addresses compatibility warnings and test failures related to legacy package handling.
- Updated major version check from 3 to 4 in package loading logic
- Modified package download compatibility check to use version 4
- Updated variable name to reflect the new version check
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
src/DynamoPackages/PackageLoader.cs | Updates major version check in package loading compatibility logic |
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs | Updates major version check and renames variable in package download logic |
Comments suppressed due to low confidence (1)
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs:953
- [nitpick] The variable name 'preDYN4Deps' is misleading. Since this filters packages with engine version < 4, it should be named 'preDYN4Deps' to 'legacyDeps' or 'incompatibleDeps' to better reflect its purpose of identifying potentially incompatible legacy packages.
var preDYN4Deps = newPackageHeaders.Where(dep => dynamoVersion.Major == 4 && Version.Parse(dep.engine_version).Major < dynamoVersion.Major);
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs
Show resolved
Hide resolved
Failing test is unrelated and it is passing locally. |
Purpose
https://jira.autodesk.com/browse/DYN-9230.
Bumping up the dynamo major check version to 4. Should fix the PackageManagerShowsWarningWhenDownloadingLegacyPackage test.
Declarations
Check these if you believe they are true
*.resx
filesRelease Notes
[DYN-9230] Bump dynamo major version check for PackageManager.
Reviewers
@DynamoDS/eidos