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

Request: Option to Output Whole File Path #88

Closed
merlinstardust opened this issue Jun 3, 2021 · 4 comments · Fixed by #89
Closed

Request: Option to Output Whole File Path #88

merlinstardust opened this issue Jun 3, 2021 · 4 comments · Fixed by #89

Comments

@merlinstardust
Copy link

Is your feature request related to a problem? Please describe.
I want to check if the file that's changed is in a specific directory.

Describe the solution you'd like

with:
  show_file_path: true

Describe alternatives you've considered
I could do git status and check within a run: block, but it seems like it would fit bettere as part of this action

Additional context
Sample output:

{
  added_files: Dockerfile src/components/Modal.jsx
  ...
}
@jackton1
Copy link
Member

jackton1 commented Jun 3, 2021

This is already supported, the full path should be returned.

Screen Shot 2021-06-03 at 1 16 31 AM

Let me know if there's a specific issue you're having

@merlinstardust
Copy link
Author

The example in the readme only shows file names. Since paths are included, I think the example needs updating and this should be explicitly mentioned in the text as well.

@jackton1
Copy link
Member

jackton1 commented Jun 3, 2021

Okay would update the README to reflect that Thanks.

@jackton1 jackton1 linked a pull request Jun 3, 2021 that will close this issue
@jackton1
Copy link
Member

jackton1 commented Jun 3, 2021

Updated https://github.com/tj-actions/changed-files#changed-files to reflect those changes.

Thanks @merlinpatt

@jackton1 jackton1 closed this as completed Jun 3, 2021
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 a pull request may close this issue.

2 participants