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

Added functionality to create new branch if target doesnt exist #33

Merged
merged 3 commits into from
Apr 18, 2020
Merged

Added functionality to create new branch if target doesnt exist #33

merged 3 commits into from
Apr 18, 2020

Conversation

RicardoMonteiroSimoes
Copy link
Contributor

I inserted the correct commands into, where I think, should be the correct place.
I was not able to test it, as I currently still havent set up docker on my computer, but if my experience and understand is right, this should work. Only thing I am not entirely sure about is the handling of failures. It might be possible to differentiate between a complete failure to clone from target and a failure due to the branch not existing.

Copy link
Owner

@andstor andstor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work 👍

I have reviewed your PR and I have some comments.

entrypoint.sh Outdated Show resolved Hide resolved
@RicardoMonteiroSimoes
Copy link
Contributor Author

Fixed a missing quotation mark, and noticed there is no need to exit if cloning fails, as that is handled already.

Copy link
Owner

@andstor andstor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch !

@andstor
Copy link
Owner

andstor commented Apr 18, 2020

I think we're good to go.

@andstor andstor merged commit 3a322de into andstor:develop Apr 18, 2020
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