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

style: Apply prettier to all code #447

Merged
merged 9 commits into from Oct 19, 2022
Merged

style: Apply prettier to all code #447

merged 9 commits into from Oct 19, 2022

Conversation

karfau
Copy link
Member

@karfau karfau commented Oct 18, 2022

and merge eslint configs as far as possible.

CodeQL discovered some new warnings after the formatting regading unused variables.
I confirmed that all of them could be either fixed or be dismissed and applied the changes as part of this PR.

Fixes #29
Fixes #130

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@karfau karfau mentioned this pull request Oct 19, 2022
1 task
@karfau karfau added this to the 0.9.0 milestone Oct 19, 2022
@karfau karfau merged commit 6601a11 into master Oct 19, 2022
@karfau karfau deleted the style/apply-prettier branch October 19, 2022 04:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

factor out test/.eslintrc.yml standard(ize) source code formatting
1 participant