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

pre-commit hook fails with an error #731

Closed
kallja opened this issue Oct 30, 2024 · 4 comments · May be fixed by #735
Closed

pre-commit hook fails with an error #731

kallja opened this issue Oct 30, 2024 · 4 comments · May be fixed by #735
Labels
stale

Comments

@kallja
Copy link
Contributor

kallja commented Oct 30, 2024

When attempting to commit using git commit, the pre-commit hook fails with the following error:
error: required option '-t, --type <type>' not specified

To reproduce run the following commands (requires docker to be installed):

  1. docker run -it --rm node:22 /bin/bash
  2. git config --global user.name "My Name" && git config --global user.email "[email protected]"
  3. git clone https://github.com/Escape-Technologies/graphql-armor.git && cd graphql-armor
  4. ./install-dev.sh
  5. touch file.md && git add file.md && git commit -m "add file.md"
@nullswan
Copy link
Contributor

@LMaxence could that be related to a mookme breaking change?

@nullswan
Copy link
Contributor

The first attempt to fix that should be going for: gookme
Otherwise, we would prefer to leverage pre-commit

@nullswan nullswan mentioned this issue Nov 8, 2024
Open
@nullswan nullswan linked a pull request Nov 8, 2024 that will close this issue
feat/gookme #735
Open
@github-actions GitHub Actions
Copy link
Contributor

This issue is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Nov 29, 2024
@github-actions GitHub Actions
Copy link
Contributor

This issue was closed because it has been inactive for 14 days since being marked as stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat/gookme Escape-Technologies/graphql-armor
2 participants
@kallja @nullswan