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

Refactor/code format #141

Merged
merged 8 commits into from
Mar 25, 2022
Merged

Refactor/code format #141

merged 8 commits into from
Mar 25, 2022

Conversation

langdal
Copy link
Member

@langdal langdal commented Mar 22, 2022

Contribution process clean up

  • Update CI pipeline
  • Add automatic code formatting (prettier)
  • Add API backend to dev container
  • Add prettier to pre-commit hook

When this PR has been merged and all other currently open PRs are merged yarn prettier --write . should be run to format all source files. If we do it before merging the other PRs we will have a lot of hard to resolve merge conflicts.

@langdal langdal marked this pull request as ready for review March 22, 2022 14:43
@langdal langdal self-assigned this Mar 22, 2022
@langdal langdal marked this pull request as draft March 22, 2022 16:25
@langdal langdal marked this pull request as ready for review March 22, 2022 16:32
@langdal langdal requested a review from j-or March 22, 2022 16:46
@langdal langdal merged commit c182116 into main Mar 25, 2022
@langdal langdal deleted the refactor/code_format branch March 25, 2022 15:19
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.

2 participants