Skip to content
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

Don't run Go tests on documentation changes #18867

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

smowton
Copy link
Contributor

@smowton smowton commented Feb 26, 2025

No description provided.

@Copilot Copilot bot review requested due to automatic review settings February 26, 2025 10:58
@smowton smowton requested a review from a team as a code owner February 26, 2025 10:58

Choose a reason for hiding this comment

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

PR Overview

This PR updates the GitHub Actions workflow to exclude running Go tests when only documentation changes occur.

  • Excludes the "go/documentation/**" directory from triggering tests on push events
  • Applies the same exclusion for pull request events

Reviewed Changes

File Description
.github/workflows/go-tests.yml Added exclusion for "go/documentation/**" paths to prevent tests from running when only documentation is changed

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

.github/workflows/go-tests.yml:17

  • [nitpick] The inserted exclusion line contains trailing whitespace. Consider removing the extra spaces for consistency.
      - "!go/documentation/**"      

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

@smowton smowton merged commit 9458f07 into main Feb 26, 2025
8 checks passed
@smowton smowton deleted the smowton/admin/avoid-pointless-test-runs branch February 26, 2025 11:34
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