Skip to content

Expand file_pattern-input to an array #205

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.

Sign up for GitHub

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

Jump to bottom
Merged
stefanzweifel merged 12 commits into from
Sep 17, 2022
Merged

Expand file_pattern-input to an array #205

stefanzweifel merged 12 commits into from
Sep 17, 2022

Conversation

@stefanzweifel
Copy link
Owner

@stefanzweifel stefanzweifel commented Feb 27, 2022 •
edited
Loading

@stefanzweifel stefanzweifel mentioned this pull request Sep 1, 2022
Closed
alexgarel
alexgarel reviewed Sep 1, 2022
View reviewed changes
alexgarel
alexgarel reviewed Sep 1, 2022
View reviewed changes
alexgarel
alexgarel reviewed Sep 1, 2022
View reviewed changes
@stefanzweifel stefanzweifel marked this pull request as ready for review September 14, 2022 18:04
@stefanzweifel
Copy link
Owner Author

stefanzweifel commented Sep 17, 2022

I've tested this change in my test repository here: stefanzweifel/git-auto-commit-action-demo-app#35

The file_pattern works much better now, but there is still an issue with globbing: If an untouched file that matches the pathspec is in the repository, the Action still fails with a Working tree clean. Nothing to commit.-mesage.
However, this can be resolved by adding disable_globbing: true to a workflow.
(Globbing is not disabled by default, due to the fear of breaking the workflow of others. See #154 for details.)

I'm gonna merge this PR to master now. A new version will be tagged after 5-6 days, if no bug reports are coming in.

@stefanzweifel stefanzweifel merged commit 4d00f10 into master Sep 17, 2022
@stefanzweifel stefanzweifel deleted the refactor/expand-file-pattern branch September 17, 2022 13:18
@stefanzweifel stefanzweifel mentioned this pull request Sep 17, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@alexgarel alexgarel alexgarel left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

changelog:changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stefanzweifel @alexgarel