Skip to content

feat(scripts): Update the publish script #41

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

Merged
merged 3 commits into from
Aug 7, 2025
Merged

Conversation

GeekMasher
Copy link
Contributor

No description provided.

@GeekMasher GeekMasher requested a review from Copilot August 7, 2025 16:40
Copy link
Contributor

@Copilot 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.

Pull Request Overview

This PR updates the publish extractor pack script to automatically detect and configure the CodeQL binary, and enables query pack creation that was previously commented out.

  • Adds automatic detection and installation of CodeQL binary (standalone or via gh extension)
  • Uncomments and updates the query pack creation commands to output to ./extractor-pack/qlpacks directory
  • Creates both library and source query packs as part of the publishing process

CODEQL_BINARY="gh codeql"
else
gh extension install github/gh-codeql
CODEQL_BINARY="gh codeql"
Copy link
Preview

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

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

If the gh-codeql extension installation fails, the script will continue executing and may fail later when trying to use $CODEQL_BINARY. Consider adding error handling to check if the installation succeeded.

Suggested change
CODEQL_BINARY="gh codeql"
if gh codeql >/dev/null; then
CODEQL_BINARY="gh codeql"
else
echo "[!] Failed to install gh-codeql extension or 'gh codeql' is not available."
exit 1
fi

Copilot uses AI. Check for mistakes.

@GeekMasher GeekMasher merged commit f5c8097 into main Aug 7, 2025
6 checks passed
@GeekMasher GeekMasher deleted the bundle-updates branch August 7, 2025 16:48
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.

1 participant