Skip to content

Conversation

@rhdh-bot
Copy link
Collaborator

Backstage release v1.44.1 has been published, this Pull Request contains the changes to upgrade bulk-import to this new release

Please review the changelog before approving, there may be manual changes needed:

Created by Version Bump 18720074374

Copy link
Member

@lholmquist lholmquist left a comment

Choose a reason for hiding this comment

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

Things tested:

  1. List of repositories and organizations correctly load based on my github/gitlab token.
  2. Created Pr's from both the "repo" list and the "org repos" list.
  3. Also edited the PR from the "add" screen which was reflected on github/gitlab
  4. Merged one of the PRs then clicked the refresh button to see that the status updated to "added".
  5. Re-looked at the repository list for the repo who's PR was merged and its status was also "added".
  6. Clicked the delete button for the other PR generated to remove it from the list, which was successful.

@sonarqubecloud
Copy link

@imykhno
Copy link
Contributor

imykhno commented Oct 24, 2025

Thank you @lholmquist for testing “open pull request” flow. We also tested "scaffolder execution" flow and all works as expected for me.

/lgtm

@dzemanov
Copy link
Member

Hi guys, so adding import '@backstage/ui/css/styles.css'; was not needed for bulk-import?

@lholmquist
Copy link
Member

It was needed but the git hook that is trigger during a commit kept changing it to import '@backstage/ui and also adding that package to the package.json. So i just left it out since it was only the development app.

If it's ok to skip that hook for just that commit, then i'll update the PR

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