Skip to content

chore: make prettier peer dependency meta#1829

Merged
alexander-akait merged 5 commits intonextfrom
chore/reduce-deps
Sep 24, 2020
Merged

chore: make prettier peer dependency meta#1829
alexander-akait merged 5 commits intonextfrom
chore/reduce-deps

Conversation

@rishabh3112
Copy link
Copy Markdown
Member

What kind of change does this PR introduce?
chore

Did you add tests for your changes?
No

If relevant, did you update the documentation?
No

Summary
Make prettier a peer dep meta as prettier is generally available to the user.

Does this PR introduce a breaking change?
People will now have to install prettier on there own to use init

Other information
Based of recent feedbacks

@rishabh3112 rishabh3112 requested a review from a team as a code owner September 24, 2020 13:00
Copy link
Copy Markdown
Member

@anshumanv anshumanv left a comment

Choose a reason for hiding this comment

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

No updates to lock file?

@rishabh3112
Copy link
Copy Markdown
Member Author

No updates to lock file?

Will do before merging.

Comment thread packages/utils/src/run-prettier.ts Outdated
Copy link
Copy Markdown
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Also please update lock file

@webpack-bot
Copy link
Copy Markdown

@rishabh3112 Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@evilebottnawi Please review the new changes.

Comment thread packages/utils/src/run-prettier.ts
Comment thread packages/utils/src/run-prettier.ts Outdated
@alexander-akait alexander-akait merged commit 5f182d8 into next Sep 24, 2020
@alexander-akait alexander-akait deleted the chore/reduce-deps branch September 24, 2020 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants