Skip to content

fix: do not commit or create PR if nothing changed#277

Merged
fjogeleit merged 1 commit intofjogeleit:masterfrom
agill17:master
Sep 9, 2021
Merged

fix: do not commit or create PR if nothing changed#277
fjogeleit merged 1 commit intofjogeleit:masterfrom
agill17:master

Conversation

@agill17
Copy link
Copy Markdown
Contributor

@agill17 agill17 commented Sep 9, 2021

Right now this action creates an empty PR/commit when there was nothing changed.
I am not 100% sure whether the === operator comparison is the best way to compare custom objects in Tsc/JS 🤷‍♂️ - but I tested on my repos and seemed functioning

@fjogeleit
Copy link
Copy Markdown
Owner

Thanks for your contribution. Looks good to me

@fjogeleit fjogeleit merged commit b3f82c2 into fjogeleit:master Sep 9, 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 this pull request may close these issues.

2 participants