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: sets up gh-actions linting and fixes pre-commit hook #19

Merged
merged 2 commits into from
Aug 30, 2023

Conversation

stianjsu
Copy link

@stianjsu stianjsu commented Aug 23, 2023

Fixes this, atleast for macOS
Screenshot 2023-08-23 at 10 54 15

pre-commit file was not created with husky add command. The filesystem flag marking the file as an executable was not set.
Fixed by running
"chmod +x ./.husky/pre-commit" from next-frontend folder

@stianjsu
Copy link
Author

Might need further testing if the flag is not persisted across devices through git. Atleast it fixed it for me

@stianjsu
Copy link
Author

stianjsu commented Aug 23, 2023

Also added linting check in next-frontend. I have seen a lot of linting errors "slipping through the cracks" in pr's

@stianjsu stianjsu changed the title fix: sets husky pre-commit file as executable fix: sets up gh-actions lintning and fixes pre-commit hook Aug 30, 2023
@stianjsu stianjsu changed the title fix: sets up gh-actions lintning and fixes pre-commit hook fix: sets up gh-actions linting and fixes pre-commit hook Aug 30, 2023
Copy link
Collaborator

@sergiosja sergiosja left a comment

Choose a reason for hiding this comment

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

Solid!

@stianjsu stianjsu merged commit 4e2bad5 into main Aug 30, 2023
@stianjsu stianjsu deleted the husky-git-hook-as-exe branch August 30, 2023 12:51
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