Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Updates CodeQL.yaml to exclude test files#3294

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
sruke merged 1 commit intomasterfromsruthi/CodeQl
Mar 11, 2025
Merged

Conversation

@sruke
Copy link
Contributor

Updates CodeQL.yaml to exclude test files

  • You've read theContributor Guide andCode of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)

Description

{Detail}

Fixes #{bug number} (in this specific format)

@srukesruke requested a review froma team as acode ownerMarch 11, 2025 16:15
@srukesruke self-assigned thisMar 11, 2025
Copy link
Contributor

CopilotAI 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 pull request updates the CodeQL.yaml configuration file to exclude test and benchmark directories without using the wildcard pattern.

  • Updated exclusion patterns for tests and benchmark directories
  • Changed exclusion pattern for docs to remove the trailing wildcard
  • Updated configuration comments in accordance with CodeQL documentation
Comments suppressed due to low confidence (2)

CodeQL.yaml:10

  • Verify that excluding 'tests' without the wildcard pattern still correctly excludes all test files, as the glob behavior may differ from expectations.
    - "tests"

CodeQL.yaml:13

  • Confirm that switching from 'docs/**' to 'docs' properly covers all documentation files, ensuring that subdirectories are excluded as intended.
    - "docs"

Copy link
Collaborator

@jennyf19jennyf19 left a comment

Choose a reason for hiding this comment

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

:shipit:

@srukesruke merged commit690cea4 intomasterMar 11, 2025
5 checks passed
@srukesruke deleted the sruthi/CodeQl branchMarch 11, 2025 17:13
This was referencedAug 1, 2025
This was referencedNov 24, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

Copilot code reviewCopilotCopilot left review comments

@jennyf19jennyf19jennyf19 approved these changes

@westin-mwestin-mwestin-m approved these changes

Assignees

@srukesruke

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sruke@jennyf19@westin-m

[8]ページ先頭

©2009-2025 Movatter.jp