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

redhat-resource-optimization: bump backstage to 1.34.2 and remove @spotify/prettier-config #2531

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

aljesusg
Copy link
Contributor

Hey, I just made a Pull Request!

bump backstage to 1.34.2 and remove @spotify/prettier-config

#2477

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • All your commits have a Signed-off-by line in the message. (more info)

@aljesusg aljesusg requested a review from 04kash January 16, 2025 09:00
@backstage-goalie
Copy link
Contributor

backstage-goalie bot commented Jan 16, 2025

Changed Packages

Package Name Package Path Changeset Bump Current Version
app workspaces/redhat-resource-optimization/packages/app none v0.0.5
backend workspaces/redhat-resource-optimization/packages/backend none v0.0.5
@backstage-community/plugin-redhat-resource-optimization-backend workspaces/redhat-resource-optimization/plugins/redhat-resource-optimization-backend minor v1.0.6
@backstage-community/plugin-redhat-resource-optimization-common workspaces/redhat-resource-optimization/plugins/redhat-resource-optimization-common minor v1.0.3
@backstage-community/plugin-redhat-resource-optimization workspaces/redhat-resource-optimization/plugins/redhat-resource-optimization minor v1.0.5

04kash
04kash previously requested changes Jan 16, 2025
Copy link
Member

@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.

Just left a comment regarding changeset

@aljesusg aljesusg requested a review from 04kash January 16, 2025 18:37
@04kash 04kash dismissed their stale review January 17, 2025 15:10

Requested changes were applied

@04kash
Copy link
Member

04kash commented Jan 17, 2025

Lgtm @aljesusg! not giving explicit approval as I'd one of the plugin owners (@christoph-jerolimov @jkilzi) to take a look as well

Copy link
Member

@christoph-jerolimov christoph-jerolimov left a comment

Choose a reason for hiding this comment

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

The code change lgtm, but I would prefer it if @jkilzi could verify that this still works. :)

Copy link
Contributor

@jkilzi jkilzi left a comment

Choose a reason for hiding this comment

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

LGTM

…otify/prettier-config

Signed-off-by: Alberto Gutierrez <[email protected]>
@04kash 04kash merged commit 613061d into backstage:main Jan 21, 2025
12 checks passed
@aljesusg aljesusg deleted the bump_rh_resource branch January 21, 2025 20:15
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.

4 participants