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.

Sign up for GitHub

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

Jump to bottom

Add yaml, md, and toml linters to pre-commit config and also update it #145

Merged
merged 3 commits into from
Aug 11, 2024
Merged

Add yaml, md, and toml linters to pre-commit config and also update it #145

merged 3 commits into from
Aug 11, 2024

Conversation

sid-maddy
Copy link
Contributor

  • Update pre-commit config
  • Run pre-commit on all files
  • Exclude repo-wide style changes from blaming

@sid-maddy sid-maddy self-assigned this Aug 11, 2024
@sid-maddy sid-maddy marked this pull request as ready for review August 11, 2024 11:21
@jonathan-d-zhang jonathan-d-zhang changed the title update pre-commit config Add yaml and toml linters to pre-commit config and also update it Aug 11, 2024
@jonathan-d-zhang jonathan-d-zhang changed the title Add yaml and toml linters to pre-commit config and also update it Add yaml, md, and toml linters to pre-commit config and also update it Aug 11, 2024
@sid-maddy
chore: update pre-commit config
bdb7c74 
Signed-off-by: Siddhesh Mhadnak <siddhesh.mhadnak@outlook.com>
@sid-maddy
style: run pre-commit on all files
4398e58 
Signed-off-by: Siddhesh Mhadnak <siddhesh.mhadnak@outlook.com>
@sid-maddy
chore(git): exclude repo-wide style changes from blaming
dfba3fb 
Signed-off-by: Siddhesh Mhadnak <siddhesh.mhadnak@outlook.com>
@sid-maddy sid-maddy merged commit dfba3fb into main Aug 11, 2024
11 checks passed
@sid-maddy sid-maddy deleted the sid/chore/pre-commit branch August 11, 2024 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonathan-d-zhang jonathan-d-zhang jonathan-d-zhang approved these changes

@AbooMinister25 AbooMinister25 Awaiting requested review from AbooMinister25

@Robin5605 Robin5605 Awaiting requested review from Robin5605

Assignees

@sid-maddy sid-maddy

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sid-maddy @jonathan-d-zhang