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] dir_names_exclude_current_dir failing to exclude #1374

Closed
3 tasks done
pl4nty opened this issue Jul 13, 2023 · 1 comment · Fixed by #1375
Closed
3 tasks done

[BUG] dir_names_exclude_current_dir failing to exclude #1374

pl4nty opened this issue Jul 13, 2023 · 1 comment · Fixed by #1375
Labels
bug Something isn't working

Comments

@pl4nty
Copy link

pl4nty commented Jul 13, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Does this issue exist in the latest version?

  • I'm using the latest release

Describe the bug?

dir_names and dir_names_exclude_current_dir are set to true, but changes in the current working directory ("/") are still present in the output.

To Reproduce

Make a root-level change with the following config:

- uses: tj-actions/changed-files@v37.1.1
  with:
    fetch_depth: 2
    skip_initial_fetch: true
    dir_names: true
    dir_names_max_depth: 1
    dir_names_exclude_current_dir: true
    files_ignore: |
      .github/**
      archive/**
    json: true

What OS are you seeing the problem on?

ubuntu-latest or ubuntu-20.04

Expected behavior?

No changes to be detected

Relevant log output

log.txt

Anything else?

https://github.com/pl4nty/containers/actions/runs/5537616198/jobs/10115639021

Code of Conduct

  • I agree to follow this project's Code of Conduct
@pl4nty pl4nty added the bug Something isn't working label Jul 13, 2023
@jackton1 jackton1 linked a pull request Jul 13, 2023 that will close this issue
@jackton1
Copy link
Member

jackton1 commented Jul 13, 2023

@pl4nty This should be resolved in the latest release.

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