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

Add command for "list-outdated" #586

Draft
wants to merge 8 commits into
base: v1-next
Choose a base branch
from
Draft

Conversation

isc-shuliu
Copy link
Collaborator

@isc-shuliu isc-shuliu commented Oct 16, 2024

Implement an outdated command mentioned in #427

@isc-kiyer
Copy link
Collaborator

@isc-shuliu this PR is related to #550 where part of running "update" should upgrade a module and run its upgrade steps (if any).

@isc-shuliu
Copy link
Collaborator Author

@isc-shuliu this PR is related to #550 where part of running "update" should upgrade a module and run its upgrade steps (if any).

Thanks for the heads-up! I'm aware of that hence posting it as a draft.

@isc-shuliu isc-shuliu changed the title Add command for "update" and "outdated" Add command for "outdated" Oct 22, 2024
@isc-kiyer isc-kiyer changed the title Add command for "outdated" Add command for "list-outdated" Oct 22, 2024
@@ -2865,6 +2878,62 @@ ClassMethod EnableIPM(ByRef pCommandInfo)
}
}

ClassMethod CanUpdate(modName As %String, semver As %IPM.General.SemanticVersion) As %Boolean
Copy link
Collaborator

Choose a reason for hiding this comment

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

@isc-shuliu Doesn't look like this method is used?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, you're right. I haven't gone to that point yet. My plan is to show if an outdated package can be updated to a particular version without violating dependency constraint. This PR is still a Work-In-Progress

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