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

add git_no_verify flag for potential hook issue #279

Merged
merged 5 commits into from
Aug 28, 2021

Conversation

tomfuertes
Copy link
Contributor

Running into issues that if eslint and prettier both run on a file my npm pre-commit hook will fail. Suggesting this but also running

If you'd like to close / logging for SEO that in the meantime I'm also just running this .yml between ci and lint-action:

      - name: Remove commit hooks
        run: rm .git/hooks/pre-commit
        run: rm .git/hooks/pre-push

action.yml Outdated Show resolved Hide resolved
@tomfuertes tomfuertes changed the title add skip_verification flag for potential hook issue add git_no_verify flag for potential hook issue Aug 9, 2021
@tomfuertes tomfuertes requested a review from ocean90 August 9, 2021 15:53
README.md Outdated Show resolved Hide resolved
@ocean90 ocean90 merged commit 466df2e into wearerequired:master Aug 28, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants