8000 [#69] Add pre-commit config by aleeusgr · Pull Request #72 · chshersh/iris · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[#69] Add pre-commit config #72

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

Merged
merged 2 commits into from
Oct 3, 2022
Merged

[#69] Add pre-commit config #72

merged 2 commits into from
Oct 3, 2022

Conversation

aleeusgr
Copy link
Contributor
@aleeusgr aleeusgr commented Oct 2, 2022

Hello!

I'm not sure this is what you had in mind, I just run pre-commit sample-config > .pre-commit-config.yaml in the root of the repo.

pre-commit run completes without errors, I'm open to suggestions on how to test and improve.

Thank you

@aleeusgr aleeusgr requested a review from chshersh as a code owner October 2, 2022 12:16
@chshersh chshersh added hacktoberfest-accepted https://hacktoberfest.com/participation/ ⚗️ dx Dev experience improvements labels Oct 3, 2022
@chshersh chshersh changed the title added pre-commit config [#69] Add pre-commit config Oct 3, 2022
Copy link
Owner
@chshersh chshersh left a comment

Choose a reason for hiding this comment

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

Thanks, looks good for the start! 🏆

@chshersh chshersh merged commit 7e153cf into chshersh:main Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚗️ dx Dev experience improvements hacktoberfest-accepted https://hacktoberfest.com/participation/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0