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.

Sign up for GitHub

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

Jump to bottom

Add prettierignore for all files #1593

Merged
merged 2 commits into from
Feb 8, 2023
Merged

Add prettierignore for all files #1593

merged 2 commits into from
Feb 8, 2023

Conversation

whoabuddy
Copy link
Contributor

Will follow up with specific settings in a separate PR, but this disables prettier for those who have it enabled on save as discussed in #1586.

prettier -c . output before:

Checking formatting...
...(truncated)...
[warn] Code style issues found in 35 files. Forgot to run Prettier?

prettier -c . output after:

Checking formatting...
All matched files use Prettier code style!

@whoabuddy
fix: add prettierignore for all files
38b665f 
Will follow up with specific settings, but this disables prettier for those who have it enabled on save.
@casey casey self-requested a review February 8, 2023 23:14
@casey
Copy link
Collaborator

casey commented Feb 8, 2023

LGTM, ty ty!

@casey casey enabled auto-merge (squash) February 8, 2023 23:15
@casey casey disabled auto-merge February 8, 2023 23:15
@casey casey enabled auto-merge (squash) February 8, 2023 23:16
@casey casey merged commit 71506a6 into ordinals:master Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@casey casey casey approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@whoabuddy @casey