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

Clearnup code using prettier #52

Closed
wants to merge 2 commits into from

Conversation

dc7303
Copy link
Member

@dc7303 dc7303 commented Aug 9, 2020

What does this PR do?

There is prettier configuration file in this project, so I am modifying the code by applying prettier to the editor.
However, some files have found that unnecessary code areas not related to commits have been modified by prettier.
It seems that some files have been modified without being applied.
I clean up code using prettier for avoid including code changes unrelated to future commits.

How should this be manually tested?

Any background context you want to provide?

What are the relevant tickets?

Fixes #

Checklist

  • Added relevant tests
  • Didn't break anything

There is `prettier` configuration file. And we need to keep using
this to maintain a unified code style.
However, it seems that some files have been modified without currently
being applied. This may contain modifications that are not related to
the content of the commit in the future.
@hackerwins
Copy link
Member

This commit was applied by #78.

@hackerwins hackerwins closed this Sep 4, 2020
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

2 participants