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 more pre commit hooks #154

Merged
merged 8 commits into from
Jul 13, 2024
Merged

Add more pre commit hooks #154

merged 8 commits into from
Jul 13, 2024

Conversation

anufdo
Copy link
Member

@anufdo anufdo commented May 26, 2024

Add following pre-commit hooks

  • check-json
  • check-xml
  • detect-private-key
  • ruby-validate
  • markdownlint
  • markdown-link-check

@anufdo anufdo requested a review from jbampton as a code owner May 26, 2024 04:08
@jbampton jbampton marked this pull request as draft May 26, 2024 21:20
jbampton
jbampton requested changes May 26, 2024
View reviewed changes
Copy link
Collaborator

@jbampton jbampton left a comment

Choose a reason for hiding this comment

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

https://github.com/NextCommunity/NextCommunity.github.io/actions/runs/9240519129/job/25420899097?pr=154

@anufdo anufdo requested a review from jbampton May 27, 2024 10:23
@anufdo anufdo marked this pull request as ready for review May 27, 2024 10:24
anufdo
anufdo commented May 28, 2024
View reviewed changes
Copy link
Member Author

@anufdo anufdo left a comment

Choose a reason for hiding this comment

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

Fixed pre commit hooks

anufdo
anufdo commented May 31, 2024
View reviewed changes
Copy link
Member Author

@anufdo anufdo left a comment

Choose a reason for hiding this comment

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

Fix broken links and markdown issues.

@jbampton jbampton self-assigned this Jul 13, 2024
@jbampton jbampton merged commit cc319d0 into NextCommunity:main Jul 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbampton jbampton jbampton approved these changes

Assignees

@jbampton jbampton

Labels
code style tests
Projects
Next
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anufdo @jbampton