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

Run prettier (existing rules, no other changes) #215

Closed
wants to merge 1 commit into from

Conversation

mikemaccana
Copy link
Collaborator

@mikemaccana mikemaccana commented Aug 24, 2023

It looks like this repo hasn't been formatted in a long time.

This PR just runs prettier. This will stop unrelated formatting changes in future PRs when files are saved & linted.

No changes to existing rules.

Alternatively: we also could just delete the rules if they're not being used. That would allow us to have regular-looking JS (2 spaces per all JS style guides) and regular looking Rust (4 spaces per rustfmt) at the same time. Prettier can't format multiple languages differently inside a markdown file yet.

Let me know what you think.

@mikemaccana
Copy link
Collaborator Author

mikemaccana commented Aug 31, 2023

Closing per @jamesrp13

Yeah I’m fine to just remove it (the prettier config) for now and maybe come back to some formatting another time

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.

None yet

1 participant