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

Support not exiting the process in case of changes in dry mode #122

Merged
merged 5 commits into from
Jul 20, 2023

Conversation

chicco785
Copy link

Fixes #80
and allows commit and push of different steps to be combined in single commit and push.

@creyD creyD changed the base branch from master to dev June 30, 2023 15:26
@creyD
Copy link
Owner

creyD commented Jun 30, 2023

@chicco785 Thank you for the bugfix and also for this new feature. I would like to change it though to allow for a more generic use of this parameter. If you have nothing against it, I have an update ready, if you give me access to your repo, I can commit it (at the moment says that I can't push to your repo.

@creyD
Copy link
Owner

creyD commented Jun 30, 2023

Or if you have nothing against it I will merge it and adjust it then, just react to this message if you are alright with me changing the code.

@chicco785
Copy link
Author

Or if you have nothing against it I will merge it and adjust it then, just react to this message if you are alright with me changing the code.

I am on vacation, sorry for being slow, feel free to change, the important thing to me it’s the functionality ;)

@creyD creyD changed the base branch from dev to feature July 1, 2023 12:40
@creyD creyD changed the base branch from feature to dev July 1, 2023 12:45
@creyD
Copy link
Owner

creyD commented Jul 1, 2023

Can't merge this at the moment, because it is not up to date with dev, would be great if you gave me the permissions (but please take your vacation first!).

@chicco785
Copy link
Author

hi @creyD,
I sent you an invitation to the repository

@creyD creyD merged commit 4bd40fc into creyD:dev Jul 20, 2023
@creyD
Copy link
Owner

creyD commented Jul 20, 2023

@chicco785 Thanks for your contribution, I will test this on DEV and later merge this to master

@chicco785
Copy link
Author

@chicco785 Thanks for your contribution, I will test this on DEV and later merge this to master

great! thanks a lot :)

@chicco785 chicco785 deleted the fix-80 branch July 20, 2023 14:53
action.yml Show resolved Hide resolved
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.

[FEATURE] modify files without pushing
2 participants