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

fix: fix and enable rustfmt linter #35

Merged
merged 9 commits into from
Feb 5, 2023

Conversation

justinchuby
Copy link
Contributor

@justinchuby justinchuby commented Feb 5, 2023

Fix the rustfmt linter because the original command would format not just the specified file but also any files it imported. Now changed to using stdin as input so that only the specified file is formatted.

Created a CI to lint this repo itself.

e.g.

image

image

@justinchuby justinchuby marked this pull request as ready for review February 5, 2023 05:18
@justinchuby justinchuby force-pushed the justinchuby/enable-self branch 2 times, most recently from 5ad3163 to 4df6a15 Compare February 5, 2023 05:28
@justinchuby justinchuby force-pushed the justinchuby/enable-self branch from beceb70 to 344557c Compare February 5, 2023 14:25
@justinchuby justinchuby force-pushed the justinchuby/enable-self branch from 88cf857 to b8f24ba Compare February 5, 2023 15:17
@justinchuby
Copy link
Contributor Author

I think it's ready

@suo
Copy link
Owner

suo commented Feb 5, 2023

love it! thanks

@suo suo merged commit 507d273 into suo:main Feb 5, 2023
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