Add Missing Languages to CodeQL Advanced Configuration#1570
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for scanning additional languages ('actions' and 'go') to the CodeQL Advanced Configuration workflow. The change transitions from scanning default languages to explicitly configuring CodeQL to scan specific languages that were previously not covered.
- Adds matrix strategy to enable scanning of 'actions' and 'go' languages
- Configures CodeQL initialization to use the matrix language parameter
| - name: Initialize CodeQL | ||
| uses: github/codeql-action/init@v3 | ||
| with: | ||
| languages: ${{ matrix.language }} |
There was a problem hiding this comment.
The indentation is inconsistent. This line should be aligned with the 'with:' parameter above using 2 spaces instead of 4 spaces to match standard GitHub Actions YAML formatting.
| languages: ${{ matrix.language }} | |
| languages: ${{ matrix.language }} |
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
|
@bkungl Do you know why the codeql check isn't running? I can't merge |
We are working to improve and streamline CodeQL configuration at GitHub. Your repository
This PR adds those languages to your configuration. Merging this PR will ensure code scanning happens in your repo reliably and should eliminate future findings in Security Findings (assuming you update the configuration as contents of your repository changes).
If your repository does not require or benefit from advanced config (e.g. for compiled languages), you can change to default setup in lieu of merging this PR.
If you have concerns or questions about this, please mention @github/prodsec-engineering in this PR or drop in our slack channel with your question.
If this configuration update will work for your repository, please approve and merge this pull request