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

[BUG] any_changed reports true without matching changed files #67

Closed
renedekat opened this issue May 16, 2021 · 4 comments · Fixed by #68
Closed

[BUG] any_changed reports true without matching changed files #67

renedekat opened this issue May 16, 2021 · 4 comments · Fixed by #68
Labels
bug Something isn't working

Comments

@renedekat
Copy link

renedekat commented May 16, 2021

Describe the bug
When testing for changes in specific file types the any_changed output is true while it should be false.

To Reproduce
Steps to reproduce the behavior:

  1. Add the following to a yml file
- name: Get specific changed files
   id: specific_changed_files
   uses: tj-actions/changed-files@v6
   with:
      files: |
        .(php|css|js|vue)$
- name: Run step if any of the listed files above change
   if: steps.specific_changed_files.outputs.any_changed == 'true'
   run: |
      echo "One or more files listed above has changed."
  1. Commit to GitHub without changing any of the match file types.

Expected behavior
echo "One or more files listed above has changed." should not run since no files are changed as can be seen in the screenshot.

Screenshots
Screenshot 2021-05-16 at 18 43 25

@renedekat renedekat added the bug Something isn't working label May 16, 2021
jackton1 added a commit that referenced this issue May 16, 2021
@jackton1
Copy link
Member

Thanks for catching this @renedekat, should be resolved here #68

jackton1 added a commit that referenced this issue May 16, 2021
@renedekat
Copy link
Author

Hello @jackton1, the issue still gives the same result with version 6.1

@jackton1
Copy link
Member

jackton1 commented May 17, 2021

@renedekat This should now be resolved in v6.2

Screen Shot 2021-05-17 at 4 34 00 AM

Thanks for testing

@renedekat
Copy link
Author

@jackton1 , thanks, I will test it later today. Many thanks for the quick responses.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants