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

chore: unify prettier to 3.4.2 #212

Merged

Conversation

mareklibra
Copy link
Contributor

@mareklibra mareklibra commented Dec 19, 2024

Unifies prettier to a single 3.4.2 version through-out the whole repo.

Different prettier versions give different results on the same sources which is causing issues/confusion when switching between local dev and CI envs.

The first commit is result of a manual change, the later one is the result of calling the prettier from the root-level and then from every workspace.

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Dec 19, 2024

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-bulk-import-backend
  • @red-hat-developer-hub/backstage-plugin-dynamic-home-page
  • @red-hat-developer-hub/backstage-plugin-catalog-backend-module-marketplace
  • @red-hat-developer-hub/backstage-plugin-marketplace-backend
  • @red-hat-developer-hub/backstage-plugin-marketplace-common
  • @red-hat-developer-hub/backstage-plugin-marketplace
  • @red-hat-developer-hub/backstage-plugin-theme

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
app workspaces/bulk-import/packages/app none v0.0.3
@red-hat-developer-hub/backstage-plugin-bulk-import-backend workspaces/bulk-import/plugins/bulk-import-backend none v5.2.1
backend workspaces/global-floating-action-button/packages/backend none v0.0.0
@red-hat-developer-hub/backstage-plugin-global-header workspaces/global-header/plugins/global-header none v0.0.0
@red-hat-developer-hub/backstage-plugin-dynamic-home-page workspaces/homepage/plugins/dynamic-home-page none v1.0.3
@red-hat-developer-hub/backstage-plugin-catalog-backend-module-marketplace workspaces/marketplace/plugins/catalog-backend-module-marketplace none v0.0.1
@red-hat-developer-hub/backstage-plugin-marketplace-backend workspaces/marketplace/plugins/marketplace-backend none v0.0.1
@red-hat-developer-hub/backstage-plugin-marketplace-common workspaces/marketplace/plugins/marketplace-common none v0.0.1
@red-hat-developer-hub/backstage-plugin-marketplace workspaces/marketplace/plugins/marketplace none v0.0.1
@red-hat-developer-hub/backstage-plugin-bc-test workspaces/theme/plugins/bc-test none v0.1.0
@red-hat-developer-hub/backstage-plugin-mui4-test workspaces/theme/plugins/mui4-test none v0.1.0
@red-hat-developer-hub/backstage-plugin-mui5-test workspaces/theme/plugins/mui5-test none v0.1.0
@red-hat-developer-hub/backstage-plugin-theme workspaces/theme/plugins/theme none v0.4.9

@mareklibra mareklibra force-pushed the FLPATH-1960.gitCommitPrettier branch from 8673a9e to ab8e255 Compare December 19, 2024 13:07
Copy link
Contributor

@gciavarrini gciavarrini left a comment

Choose a reason for hiding this comment

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

lgtm

@mareklibra mareklibra force-pushed the FLPATH-1960.gitCommitPrettier branch from ab8e255 to b2797f4 Compare December 19, 2024 13:17
@gciavarrini gciavarrini self-requested a review December 19, 2024 13:26
Copy link
Contributor

@gciavarrini gciavarrini left a comment

Choose a reason for hiding this comment

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

ignore all the generated files and revert the changes

@mareklibra mareklibra force-pushed the FLPATH-1960.gitCommitPrettier branch 9 times, most recently from c657a3a to 91f9285 Compare December 21, 2024 21:01
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.

Hi @mareklibra, this looks much better now. But can you take a look into the root package.json and yarn.lock?

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@mareklibra
Copy link
Contributor Author

I don't see a commit adding those changes.
I will rebase due to merge conflicts, I guess those changes will disappear then.

@mareklibra mareklibra force-pushed the FLPATH-1960.gitCommitPrettier branch 2 times, most recently from a53813f to c3836f6 Compare January 8, 2025 08:45
@mareklibra mareklibra force-pushed the FLPATH-1960.gitCommitPrettier branch from c3836f6 to 8b8a18a Compare January 8, 2025 08:50
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.

Thanks for this PR and cleaning it up again and again 👍

Lgtm!

Copy link
Contributor

@gciavarrini gciavarrini left a comment

Choose a reason for hiding this comment

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

lgtm

@mareklibra mareklibra merged commit e2f788f into redhat-developer:main Jan 13, 2025
24 checks passed
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