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 on TypeScript test files #1812

Merged
merged 4 commits into from
Sep 26, 2022

Conversation

niklasmohrin
Copy link
Member

@niklasmohrin niklasmohrin commented Sep 26, 2022

No description provided.

This way, the tests will also be formatted / checked. We could also
format all the files in `evap/static/ts` which would include
configuration files, but that would mean to add a `.prettierignore` to
the root of the project (or always including `--ignore-path`) and I
don't like that.
@niklasmohrin niklasmohrin marked this pull request as ready for review September 26, 2022 19:28
@niklasmohrin niklasmohrin merged commit ed285c5 into e-valuation:main Sep 26, 2022
@niklasmohrin niklasmohrin deleted the prettier-tests branch September 26, 2022 22:11
@niklasmohrin
Copy link
Member Author

@he3lixxx Using "Merge commit" merging on gh worked with .git-blame-ignore-revs, no second PR needed 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants