Skip to content

Conversation

@felickz
Copy link
Contributor

@felickz felickz commented Feb 3, 2026

Removed support for several programming languages in CodeQL workflow and added a custom config for paths.

Scanning the dist folder is not productive:

image image image

Removed support for several programming languages in CodeQL workflow and added a custom config for paths.
Copilot AI review requested due to automatic review settings February 3, 2026 22:50
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/checkout 4.*.* 🟢 6.4
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained🟢 57 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 5
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Packaging⚠️ -1packaging workflow not detected
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
Branch-Protection🟢 6branch protection is not maximal on development and all release branches
Vulnerabilities🟢 73 existing vulnerabilities detected
SAST🟢 8SAST tool detected but not run on all commits
actions/github/codeql-action/analyze 4.*.* UnknownUnknown
actions/github/codeql-action/init 4.*.* UnknownUnknown

Scanned Files

  • .github/workflows/codeql.yml

Copy link

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 CodeQL workflow to focus on relevant code analysis by configuring language support and scan paths. The workflow now specifically analyzes GitHub Actions and JavaScript/TypeScript code while excluding the dist folder from scans to avoid unproductive analysis of generated/compiled code.

Changes:

  • Limited language scanning to actions and javascript-typescript only
  • Added custom configuration to restrict scans to the src directory
  • Excluded dist and tests folders from analysis

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

with:
languages: ${{ matrix.language }}
build-mode: ${{ matrix.build-mode }}
# Ignore /dist /tests
Copy link

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

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

The comment indicates that both /dist and /tests should be ignored, but the configuration only excludes these directories by scanning the src directory. If the intention is to also exclude tests from scanning, this should be explicitly documented that the src path implicitly excludes them, or the comment should be updated to reflect the actual configuration approach.

Suggested change
# Ignore /dist /tests
# Limit analysis to the src directory (other top-level folders such as dist/tests are not scanned)

Copilot uses AI. Check for mistakes.
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