Skip to content

Fix %IPM.Storage.QualifiedModuleInfo.%OnNew()#1103

Merged
isc-cborbonm merged 14 commits intomainfrom
qualifiedModInfo
Mar 17, 2026
Merged

Fix %IPM.Storage.QualifiedModuleInfo.%OnNew()#1103
isc-cborbonm merged 14 commits intomainfrom
qualifiedModInfo

Conversation

@isc-cborbonm
Copy link
Copy Markdown
Collaborator

@isc-cborbonm isc-cborbonm commented Mar 13, 2026

Fixes #1102: %IPM.Storage.QualifiedModuleInfo.%OnNew() will now copy over version related properties

Carolina Borbon Miranda and others added 2 commits March 13, 2026 13:08
Fix issue with version properties copying in %IPM.Storage.QualifiedModuleInfo:%New()
@isc-cborbonm isc-cborbonm self-assigned this Mar 13, 2026
@isc-cborbonm isc-cborbonm added the bug Something isn't working label Mar 13, 2026
Copy link
Copy Markdown
Collaborator

@isc-dchui isc-dchui left a comment

Choose a reason for hiding this comment

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

Please tag the issue in the description using the magic closing words!

Comment thread CHANGELOG.md Outdated
Copy link
Copy Markdown
Collaborator

@isc-eneil isc-eneil left a comment

Choose a reason for hiding this comment

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

Left a couple comments @isc-cborbonm !

Comment thread CHANGELOG.md Outdated
Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls Outdated
Comment thread CHANGELOG.md Outdated
Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls Outdated
Copy link
Copy Markdown
Collaborator

@isc-jili isc-jili left a comment

Choose a reason for hiding this comment

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

Looks great @isc-cborbonm ! Left some small comments

Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls
Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls Outdated
Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls
Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls Outdated
@isc-cborbonm isc-cborbonm requested a review from isc-jili March 17, 2026 14:50
Comment thread tests/integration_tests/Test/PM/Integration/QualifiedModInfo.cls Outdated
@isc-cborbonm isc-cborbonm requested a review from isc-eneil March 17, 2026 15:26
@isc-cborbonm isc-cborbonm merged commit 4fa6b26 into main Mar 17, 2026
24 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

%IPM.Repo.Manager:SearchRepositoriesForModule() discards AllVersions

5 participants