Skip to content

[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

Merged
merged 4 commits into from
Jul 24, 2025

Conversation

reddyashish
Copy link
Contributor

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

  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB
  • This PR introduces new feature code involve network connecting and is tested with no-network mode.

Release Notes

[DYN-9230] Bump dynamo major version check for PackageManager.

Reviewers

@DynamoDS/eidos

Copy link

@github-actions github-actions bot left a 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

@zeusongit zeusongit added this to the 3.6 milestone Jul 23, 2025
@zeusongit zeusongit removed this from the 3.6 milestone Jul 23, 2025
@QilongTang QilongTang requested a review from Copilot July 23, 2025 20:06
Copy link
Contributor

@Copilot Copilot AI left a 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);

@reddyashish
Copy link
Contributor Author

Failing test is unrelated and it is passing locally.

@reddyashish reddyashish merged commit 076d7de into DynamoDS:master Jul 24, 2025
26 of 28 checks passed
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.

4 participants