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

Linting failing on main branch after PR merge commits #14

Closed
alexanderianblair opened this issue Jul 18, 2024 · 1 comment
Closed

Linting failing on main branch after PR merge commits #14

alexanderianblair opened this issue Jul 18, 2024 · 1 comment

Comments

@alexanderianblair
Copy link
Contributor

The Lint workflow is currently failing on the main branch after being triggered by a PR merge commit, with the following error:

  2024-07-17 10:26:17 [INFO]   --------------------------------------------
  2024-07-17 10:26:17 [INFO]   Gathering GitHub information...
  2024-07-17 10:26:17 [INFO]   Successfully validated GITHUB_WORKSPACE: /github/workspace
  2024-07-17 10:26:17 [INFO]   Successfully found GITHUB_EVENT_PATH: /github/workflow/event.json]
  2024-07-17 10:26:17 [INFO]   Successfully found GITHUB_SHA: e9b88d3995fb6e4c701f4f5bb2dd90d52c49f761
  2024-07-17 10:26:17 [INFO]   Successfully found GITHUB_PUSH_COMMIT_COUNT: 20
  fatal: ambiguous argument 'HEAD~20': unknown revision or path not in the working tree.
  Use '--' to separate paths from revisions, like this:
  'git <command> [<revision>...] -- [<file>...]'
  2024-07-17 10:26:17 [FATAL]   Failed to initialize GITHUB_BEFORE_SHA for a push event. Output: HEAD~20
  /action/lib/linter.sh: line 720: SUPER_LINTER_PRIVATE_OUTPUT_DIRECTORY_PATH: unbound 

Attempts to fix by changing the version of Super-Linter used in the workflow, in #12 and #13, have run the Lint workflow successfully on merge to main, but have failed on all subsequent merge commits.

The Lint workflow does appear to run correctly in PRs, however, and is providing useful checks.

@alexanderianblair
Copy link
Contributor Author

Closing as this now seems to have been resolved following the update to Super Linter v6.7.0

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

No branches or pull requests

1 participant