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

Changes to testdata and templates don't run CI tests #4067

Open
glasser opened this issue Dec 17, 2023 · 0 comments
Open

Changes to testdata and templates don't run CI tests #4067

glasser opened this issue Dec 17, 2023 · 0 comments
Labels
bug Something isn't working Stale

Comments

@glasser
Copy link
Contributor

glasser commented Dec 17, 2023

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Overview of the Issue

Changes to testdata and *.tmpl templates do not trigger PR CI test runs, but they can make tests fail.

Reproduction Steps

I think the path filter in test.yml should notice changes to templates and testdata at the very least:

  changes:
    outputs:
      should-run-tests: ${{ steps.changes.outputs.go == 'true' }}
    if: github.event.pull_request.draft == false
    runs-on: ubuntu-22.04
    steps:
      - uses: actions/checkout@v4
      - uses: dorny/paths-filter@v2
        id: changes
        with:
          filters: |
            go:
              - '**.go'
              - 'go.*'
              - '.github/workflows/test.yml'
@glasser glasser added the bug Something isn't working label Dec 17, 2023
@dosubot dosubot bot added the Stale label Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Stale
Projects
None yet
Development

No branches or pull requests

1 participant