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

Update to latest prettier-plugin-sort-imports and removed dummy package #5279

Merged
merged 1 commit into from
Feb 24, 2023

Conversation

acelaya
Copy link
Contributor

@acelaya acelaya commented Feb 24, 2023

Version 4.1 of @trivago/prettier-plugin-sort-imports fixes a peer dependency which was not properly marked as optional, so we no longer need the dummy local package replacing it.

@acelaya acelaya requested a review from robertknight February 24, 2023 13:42
@acelaya
Copy link
Contributor Author

acelaya commented Feb 24, 2023

I'm not sure what's going on with formatting.

This happened to me at first, but then downgraded and upgraded again and that seemed to solve the issue, so I pushed the branch. (edit: We run prettier with --cache, that's why it seemed to work)

Here it seems to be even worse, as it's marking a bunch of files as incorrectly formatted.

@acelaya
Copy link
Contributor Author

acelaya commented Feb 24, 2023

Issue reported trivago/prettier-plugin-sort-imports#211.

Putting this back to draft for now.

@acelaya acelaya marked this pull request as draft February 24, 2023 14:27
@acelaya acelaya force-pushed the remove-dummy-module branch from 32d1a24 to f4688a8 Compare February 24, 2023 15:23
@acelaya acelaya marked this pull request as ready for review February 24, 2023 15:26
@codecov
Copy link

codecov bot commented Feb 24, 2023

Codecov Report

Merging #5279 (f4688a8) into main (adc5bad) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #5279   +/-   ##
=======================================
  Coverage   99.18%   99.18%           
=======================================
  Files         235      235           
  Lines        9342     9342           
  Branches     2239     2239           
=======================================
  Hits         9266     9266           
  Misses         76       76           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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.

2 participants