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

Add prettier #7

Closed
wants to merge 3 commits into from
Closed

Conversation

stex-docto
Copy link
Contributor

fix #6

@michelgrootjans
Copy link
Owner

This is a big improvement, thank you. However, there are places where I don't agree with prettier, like for example application/domain/initial-state.js.

What are the options to be lenient in some places?

@stex-docto
Copy link
Contributor Author

I'll look into it. 👍

@stex-docto stex-docto closed this Aug 2, 2023
@stex-docto stex-docto force-pushed the add-prettier branch 2 times, most recently from 94462ea to 8080260 Compare August 2, 2023 10:15
@stex-docto stex-docto reopened this Aug 2, 2023
@@ -1,16 +1,16 @@
// prettier-ignore
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is how you disable prettier for the next instruction only.

@stex-docto
Copy link
Contributor Author

I've put 120 char as line-width and disable formatting on this specific files after refomatting by end to longest line.

@michelgrootjans
Copy link
Owner

Has been merged with #9

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.

Code formatter
2 participants