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(janus-idp/cli): fixes issue with missing configSchema.json #2368

Merged
merged 2 commits into from
Oct 17, 2024

Conversation

kadel
Copy link
Member

@kadel kadel commented Oct 16, 2024

The problem was that plugins in @backstage-community were accidentally ignored when searching for config schemas because startsWith('@backstage') was used as a main filtering rule.

So, when the @backstage-community namespace was introduced and plugins were migrated to it, it started to ignore them.

This simple fix ensures that only packages in the core @backstage can be filtered out.

Fixes: https://issues.redhat.com/browse/RHIDP-3574

@kadel kadel requested a review from a team as a code owner October 16, 2024 12:14
Copy link

changeset-bot bot commented Oct 16, 2024

🦋 Changeset detected

Latest commit: 5a1e2f9

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

@kadel kadel force-pushed the fix-configSchema branch 2 times, most recently from 83f3e5b to 4fb58c9 Compare October 16, 2024 12:35
This was affecting packages in @backstage-community
@kadel kadel requested a review from a team as a code owner October 16, 2024 12:37
@openshift-ci openshift-ci bot removed the lgtm label Oct 17, 2024
Copy link

sonarcloud bot commented Oct 17, 2024

@openshift-ci openshift-ci bot added the lgtm label Oct 17, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 140cc50 into janus-idp:main Oct 17, 2024
10 checks passed
@kadel
Copy link
Member Author

kadel commented Oct 22, 2024

/cherrypick release-1.3

@openshift-cherrypick-robot

@kadel: new pull request created: #2411

In response to this:

/cherrypick release-1.3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

04kash pushed a commit to 04kash/backstage-plugins that referenced this pull request Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants