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

fix(cli): extend --ignore-version-check [release-1.4] #2619

Conversation

openshift-cherrypick-robot

This is an automated cherry-pick of #2616

/assign gashcrumb

This change updates the cli's --ignore-version-check flag to also ignore
the check performed on a embedded transitive dependencies that are moved
to peer dependencies, when it is known that the mismatch is runtime
compatible.

Signed-off-by: Stan Lewis <[email protected]>
Copy link

changeset-bot bot commented Jan 20, 2025

🦋 Changeset detected

Latest commit: 949e8b5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@janus-idp/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@nickboldt nickboldt changed the title [release-1.4] fix(cli): extend --ignore-version-check fix(cli): extend --ignore-version-check [release-1.4] Jan 20, 2025
@openshift-ci openshift-ci bot added the lgtm label Jan 21, 2025
@kadel kadel removed the lgtm label Jan 21, 2025
@nickboldt
Copy link
Member

nickboldt commented Jan 21, 2025

Stan said:

oh yes, that can be merged

y'all should just mash the merge button or add the lgtm label - saves time :D

@nickboldt nickboldt added the lgtm label Jan 21, 2025
@nickboldt nickboldt merged commit 374eef5 into janus-idp:release-1.4 Jan 21, 2025
10 of 12 checks passed
@kadel
Copy link
Member

kadel commented Jan 21, 2025

I reviewed this, it look ok. But than I realized that I don't fully understand how the backporting is supposed to work with plugins 😇 So I got scared and removed the lgtm label :-D

@kadel
Copy link
Member

kadel commented Jan 21, 2025

But now I undertsand it as I see #2621 correctly pusghing new 1.18.x version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants