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

🔧 Repository: Remove Janus IDP cli from multiple plugins #1789

Closed
12 of 13 tasks
christoph-jerolimov opened this issue Oct 31, 2024 · 3 comments
Closed
12 of 13 tasks
Assignees
Labels
bug Something isn't working

Comments

@christoph-jerolimov
Copy link
Member

christoph-jerolimov commented Oct 31, 2024

📜 Description

When we, as Red Hat, contributed some of our backstage plugins to the community, we included a dependency to @janus-idp/cli and used this in the build process (the yarn export-dynamic scripts).

We want remove that dependency and align the build process with other plugins.

For this we want remove this dependency from this workspaces:

👍 Expected behavior

No @janus-idp/cli dependency and export-dynamic script in any workspace.

👎 Current Behavior

@janus-idp/cli dependency and export-dynamic script in some workspaces.

📃 Provide the context for the Bug.

The script was used to add additional build artifacts (the dist-scalprum folder) to the npm package:

https://www.npmjs.com/package/@backstage-community/plugin-redhat-argocd/v/1.10.0?activeTab=code

👀 Have you spent some time to check if this bug has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

Are you willing to submit PR?

Yes I am willing to submit a PR!

@04kash
Copy link
Contributor

04kash commented Nov 5, 2024

@christoph-jerolimov can we have a checklist of all steps needed for this issue, this is what I'm thinking:

  • Delete the dist-dynamic folder.
  • In package.json, remove the export-dynamic and postinstall scripts, along with any other scripts referencing export-dynamic.
  • Remove @janus-idp/cli from dependencies.
  • Remove "dist-dynamic/", "dist-dynamic/dist/", and "dist-scalprum" entries from the files field in package.json.
  • Rename app-config.yaml to app-config-example.yaml.

@christoph-jerolimov
Copy link
Member Author

@04kash That's the correct steps. My goal is to have that PRs for all the workspaces in the list above up by today.

@christoph-jerolimov
Copy link
Member Author

christoph-jerolimov commented Nov 7, 2024

We might or might not cleanup also the redhat-argocd plugin. There are some internal discussions about the timing. So it might take some more time, but we can close this tracker already now.

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

No branches or pull requests

2 participants