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

feat(janus-idp/cli): package-dynamic-plugins update annotations #2609

Merged
merged 4 commits into from
Jan 9, 2025

Conversation

kadel
Copy link
Member

@kadel kadel commented Jan 9, 2025

fixes https://issues.redhat.com/browse/RHIDP-5290

  • rename com.redhat.rhdh.plugins to io.backstage.dynamic-packages to
    make it more "upstream friendly"
  • add io.backstage.marketplace/<plugin-name> annotations, value is read
    from file specified by optional --marketplace flag
  • use base64 for annotation values

kadel added 2 commits January 9, 2025 13:28
- rename com.redhat.rhdh.plugins to io.backstage.dynamic-packages to
  make it more "upstream friendly"
- add io.backstage.marketplace/<plugin-name> annotations, value is read
  from file specified by optional --marketplace flag
- use base64 for anotation values

Signed-off-by: Tomas Kral <[email protected]>
@kadel kadel requested review from davidfestal, jesuino, gashcrumb and a team as code owners January 9, 2025 12:34
Copy link

changeset-bot bot commented Jan 9, 2025

🦋 Changeset detected

Latest commit: 2a3acd4

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 Major

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

Copy link

sonarqubecloud bot commented Jan 9, 2025

@openshift-ci openshift-ci bot added the lgtm label Jan 9, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit eac5451 into janus-idp:main Jan 9, 2025
9 checks passed
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.

2 participants