Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

docs(tip): add package.json tip to improve a prettier setup #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

filiphsps
Copy link

@filiphsps filiphsps commented Oct 21, 2023

Let's try that again, apparently GitHub weren't too thrilled about me renaming the branch of an active PR. I'll fixup the build lengthen the description when I get back to a terminal later, doesn't hurt to get it under a few eyes a bit earlier though!


Seen a lot of people get this wrong, so let's add a tip explaining the benefits of setting the prettier configuration in package.json rather than .prettierrc or prettier.json.

@filiphsps filiphsps requested a review from KyleTryon as a code owner October 21, 2023 09:00
@netlify
Copy link

netlify bot commented Oct 21, 2023

Deploy Preview for configtips ready!

Name Link
🔨 Latest commit 0e0786a
🔍 Latest deploy log https://app.netlify.com/sites/configtips/deploys/65339837b70d0300073dd9a5
😎 Deploy Preview https://deploy-preview-68--configtips.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@filiphsps filiphsps force-pushed the prettier-in-package-json branch from 3f4d7a2 to 1aba98d Compare October 21, 2023 09:19
Seen a lot of people get this wrong, so let's add a tip explaining
the benefits of setting the prettier configuration in `package.json`
rather than `.prettierrc` or `prettier.json`.
@filiphsps filiphsps force-pushed the prettier-in-package-json branch from 1aba98d to 0e0786a Compare October 21, 2023 09:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant