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

plugin(quay-actions): migration of quay-actions plugin to backstage/community-plugins workspace #1897

Closed

Conversation

Fortune-Ndlovu
Copy link
Contributor

@Fortune-Ndlovu Fortune-Ndlovu commented Nov 8, 2024

Hey, I just made a Pull Request!

The quay-actions plugin from the janus-idp/backstage-plugins repository was migrated to the community plugins, The migration was performed by following the manual migration steps outlined in the Community Plugins CONTRIBUTING guide

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

@backstage-goalie
Copy link
Contributor

backstage-goalie bot commented Nov 8, 2024

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage-community/plugin-scaffolder-backend-module-quay workspaces/quay/plugins/quay-actions patch v2.2.1
@backstage-community/cli workspaces/repo-tools/packages/cli none v0.0.3

@04kash
Copy link
Contributor

04kash commented Nov 8, 2024

Running yarn backstage-cli repo fix --publish from the workspace should fix the CI issue

@Fortune-Ndlovu
Copy link
Contributor Author

Of course, yarn backstage-cli repo fix --publish will actually fix and synchronize the workspace directly while yarn backstage-cli repo fix --check --publish will only check for inconsistencies without making any changes.

@04kash
Copy link
Contributor

04kash commented Nov 8, 2024

Can you also rename the workspace to just quay instead of quay actions? We can later add other quay plugins in there as well then

@04kash
Copy link
Contributor

04kash commented Nov 8, 2024

I think you renamed the wrong directory 😅, instead of renaming workspaces/quay-actions you seemed to have renamed workspaces/quay-actions/plugins/quay-actions

workspaces/quay/package.json Outdated Show resolved Hide resolved
@04kash
Copy link
Contributor

04kash commented Nov 8, 2024

Thanks, you'd also have to run yarn build:api-reports after this change, as the reports should be named report.api.md and not api-report.md

@Fortune-Ndlovu
Copy link
Contributor Author

Thanks, @04kash, @awanlin!

Copy link
Contributor

@04kash 04kash left a comment

Choose a reason for hiding this comment

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

looks good!

Copy link
Contributor

@awanlin awanlin left a comment

Choose a reason for hiding this comment

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

Thanks for contributing this plugin @Fortune-Ndlovu, left a few easy comments.

There is one additional task that would be great to see is the in app examples. Here are the ones for the GitHub scaffolder actions: https://github.com/backstage/backstage/blob/master/plugins/scaffolder-backend-module-github/src/actions/github.examples.ts. These then show up in the app here: https://demo.backstage.io/create/actions

workspaces/quay/.changeset/metal-fishes-cover.md Outdated Show resolved Hide resolved
workspaces/quay/package.json Show resolved Hide resolved
workspaces/quay/package.json Show resolved Hide resolved
workspaces/quay/package.json Show resolved Hide resolved
workspaces/quay/plugins/quay-actions/turbo.json Outdated Show resolved Hide resolved
workspaces/quay/plugins/quay-actions/tsconfig.json Outdated Show resolved Hide resolved
@04kash
Copy link
Contributor

04kash commented Nov 12, 2024

Can you also add this workspace to the CODEOWNERS file?

@04kash 04kash mentioned this pull request Nov 12, 2024
5 tasks
@backstage-goalie
Copy link
Contributor

Thanks for the contribution!
All commits need to be DCO signed before they are reviewed. Please refer to the the DCO section in CONTRIBUTING.md or the DCO status for more info.

Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Fortune-Ndlovu and others added 8 commits November 12, 2024 17:40
Signed-off-by: Fortune-Ndlovu <fndlovu@redhat.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
* repo-tools: update @backstage/repo-tools version

Signed-off-by: Kashish Mittal <kmittal@redhat.com>

* Update workspaces/repo-tools/packages/cli/src/lib/workspaces/templates/workspace/package.json.hbs

Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com>
Signed-off-by: Kashish Mittal <113269381+04kash@users.noreply.github.com>

---------

Signed-off-by: Kashish Mittal <kmittal@redhat.com>
Signed-off-by: Kashish Mittal <113269381+04kash@users.noreply.github.com>
Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
* Remove private flag from Mend

Signed-off-by: Darius <dariusz.sobkowicz@gmail.com>

* Add changeset

Signed-off-by: Darius <dariusz.sobkowicz@gmail.com>

* Fix versioning

Signed-off-by: Darius <dariusz.sobkowicz@gmail.com>

* Update changeset

Signed-off-by: Darius <dariusz.sobkowicz@gmail.com>

* Add codeowner

Signed-off-by: Darius <dariusz.sobkowicz@gmail.com>

---------

Signed-off-by: Darius <dariusz.sobkowicz@gmail.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
Signed-off-by: Vincenzo Scamporlino <vincenzos@spotify.com>
Signed-off-by: Fortune-Ndlovu <ndlovufortune97@gmail.com>
@Fortune-Ndlovu
Copy link
Contributor Author

the noise came from:
➜ git rebase HEAD~25 --signoff
➜ git push --force-with-lease origin migrate-quay-actions

@04kash 04kash mentioned this pull request Nov 12, 2024
5 tasks
@04kash
Copy link
Contributor

04kash commented Nov 13, 2024

@Fortune-Ndlovu the PR seems to have some unrelated commits 🤔

Copy link
Contributor

@awanlin awanlin left a comment

Choose a reason for hiding this comment

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

Thanks for the updates, spotted to small items to fix. 👍

workspaces/quay/.changeset/metal-fishes-cover.md Outdated Show resolved Hide resolved
@awanlin
Copy link
Contributor

awanlin commented Nov 14, 2024

Hi @Fortune-Ndlovu, something odd has happened to this PR that seems like it may have reverted a number of your changes to address comments. Can you give it a look, please?

@schultzp2020
Copy link
Contributor

Hi @Fortune-Ndlovu, something odd has happened to this PR that seems like it may have reverted a number of your changes to address comments. Can you give it a look, please?

It appears that you force-pushed without including the changes made directly in this PR through committed suggestions.

@Fortune-Ndlovu
Copy link
Contributor Author

Rebase has made a mess on this PR, I am creating a new PR now with the relevant suggestions. Should take a few minutes, thanks.

@Fortune-Ndlovu Fortune-Ndlovu closed this by deleting the head repository Nov 14, 2024
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.

9 participants