Skip to content

Check GIT_COMMIT availability before falling back to CLI#153

Open
lindluni wants to merge 8 commits into
mainfrom
git-commit-fallback
Open

Check GIT_COMMIT availability before falling back to CLI#153
lindluni wants to merge 8 commits into
mainfrom
git-commit-fallback

Conversation

@lindluni
Copy link
Copy Markdown
Contributor

This pull request makes a small improvement to the ExecuteCodeQL.groovy script by enhancing the logic for determining the commit hash when uploading SARIF files.

Enhancements to commit determination:

  • jenkins/shared-libraries/linux/vars/ExecuteCodeQL.groovy: Added a conditional check to prioritize the GIT_COMMIT environment variable for determining the commit hash. If GIT_COMMIT is not set, the script falls back to using git rev-parse HEAD. This ensures more robust and explicit commit identification during SARIF file uploads.

@lindluni lindluni requested a review from Copilot April 24, 2025 00:52
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • jenkins/shared-libraries/linux/vars/ExecuteCodeQL.groovy: Language not supported

@lindluni lindluni force-pushed the git-commit-fallback branch from 5acebb1 to 5a7c8c1 Compare April 24, 2025 01:01
lindluni and others added 3 commits May 23, 2025 10:20
Signed-off-by: Brett Logan <lindluni@github.com>
Signed-off-by: Brett Logan <lindluni@github.com>
Signed-off-by: Brett Logan <9400927+lindluni@users.noreply.github.com>
@lindluni lindluni force-pushed the git-commit-fallback branch from c2dc8ae to 9af3628 Compare May 23, 2025 14:21
Signed-off-by: Brett Logan <9400927+lindluni@users.noreply.github.com>
@lindluni lindluni force-pushed the git-commit-fallback branch from d8ee9b5 to 16e726e Compare May 23, 2025 14:29
lindluni added 4 commits May 23, 2025 10:36
Signed-off-by: Brett Logan <lindluni@github.com>
Signed-off-by: Brett Logan <lindluni@github.com>
Signed-off-by: Brett Logan <lindluni@github.com>
Signed-off-by: Brett Logan <lindluni@github.com>
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