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

Use find instead of git diff #71

Merged
merged 1 commit into from
Dec 21, 2022
Merged

Use find instead of git diff #71

merged 1 commit into from
Dec 21, 2022

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 21, 2022

📝 Brief description

Use find instead of git diff

💻 Commits

  • dc9b051 - Krzysztof Szyper - 2022-12-21 19:36:07
    Use find instead of git diff
    Using find . -newer ... should give the same results as git diff in dockerized environment (minus M at beginning of the line).
    While still giving empty string when no files where modified.
    This will prevent error: unknown option `cached'.

    Bump release to v0.3.2

    Second finding in git not handling unchanged files properly #69

📁 Modified files

Makefile | 2 +-
README.md | 6 +++---
action.yml | 2 +-
entrypoint.sh | 3 ++-
4 files changed, 7 insertions(+), 6 deletions(-)

⚠️ Additional information

  • Pushed to a branch with a proper name and provided proper commit message.
  • Provided a clear and concise description of what the issue is.

Check CONTRIBUTING.md and CODE_OF_CONDUCT.md for more information

@github-actions github-actions bot requested review from ChristophShyper and a team as code owners December 21, 2022 18:35
@github-actions github-actions bot added the bugfix Bug needing a fix label Dec 21, 2022
Using `find . -newer ...` should give the same results as `git diff` in dockerized environment (minus `M` at beginning of the line).
While still giving empty string when no files where modified.
This will prevent ``error: unknown option `cached'``.

Bump release to v0.3.2

Second finding in #69
@ChristophShyper ChristophShyper merged commit e66b042 into master Dec 21, 2022
@ChristophShyper ChristophShyper deleted the bugfix/find-git branch December 21, 2022 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bug needing a fix
Development

Successfully merging this pull request may close these issues.

1 participant