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

Auto-format cjs, cts, mts, jsonc, json5, yaml #260

Merged
merged 3 commits into from
Oct 6, 2022

Conversation

@lgtm-com
Copy link

lgtm-com bot commented Aug 26, 2022

This pull request introduces 1 alert when merging 135fc70 into 46e2d2a - view on LGTM.com

new alerts:

  • 1 for Syntax error

@alecmev alecmev force-pushed the patch-2 branch 3 times, most recently from 3a99851 to c532ded Compare August 26, 2022 20:04
@jonlabelle
Copy link
Owner

jonlabelle commented Aug 31, 2022

Hi @alecmev. Thanks for the PR!

Can you address the CI/Flake 8 issues?

@alecmev
Copy link
Contributor Author

alecmev commented Aug 31, 2022

Hi, done!

@jonlabelle
Copy link
Owner

Thanks @alecmev!

@jonlabelle jonlabelle merged commit 5862c81 into jonlabelle:master Oct 6, 2022
@alecmev alecmev deleted the patch-2 branch October 7, 2022 08:02
@alecmev
Copy link
Contributor Author

alecmev commented Oct 7, 2022

Thank you!

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.

None yet

2 participants