-
Notifications
You must be signed in to change notification settings - Fork 92
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
Introduce Prettier #44
Conversation
a2b8ad2
to
3edeca6
Compare
src/document/document.ts
Outdated
logger.trivial(changes.map((change) => | ||
`${change.getID().getAnnotatedString()}\t${change.getAnnotatedString()}` | ||
).join('\n')); | ||
logger.trivial( | ||
changes | ||
.map( | ||
(change) => | ||
`${change | ||
.getID() | ||
.getAnnotatedString()}\t${change.getAnnotatedString()}`, | ||
) | ||
.join('\n'), | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems harder to read. We need to find a way to prevent a newline per chained call.
https://eslint.org/docs/rules/newline-per-chained-call#ignorechainwithdepth
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3edeca6
to
c5f975f
Compare
c5f975f
to
4df5ed1
Compare
* Apply prettier * Change code style of Prettier Co-authored-by: hyunwoo.jo <hyunwoo.jo@navercorp.com>
What does this PR do?
Change the code style of Prettier
How should this be manually tested?
Any background context you want to provide?
What are the relevant tickets?
Fixes #41
Checklist