Skip to content

Conversation

@GeekMasher
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings September 12, 2025 09:35
@GeekMasher GeekMasher requested a review from a team as a code owner September 12, 2025 09:35
@github-actions
Copy link

Dependency Review

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

Snapshot Warnings

⚠️: No snapshots were found for the head SHA fc0fd7f.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

Copy link
Contributor

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 adds a method to determine the appropriate CodeQL directory for the action, with improved path resolution to avoid indexing issues. The change modifies the directory selection logic to use parent directories and canonicalized paths.

  • Updates get_codeql_dir() method to use parent of working directory instead of working directory itself
  • Adds path canonicalization to resolve symbolic links and relative paths
  • Enhances documentation to explain the rationale for using parent directories

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@GeekMasher GeekMasher merged commit bade8f2 into main Sep 12, 2025
10 checks passed
@GeekMasher GeekMasher deleted the output-location branch September 12, 2025 09:49
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