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

Fixed bugs with adding URL at specific commit #95

Merged
merged 5 commits into from
Feb 2, 2022

Conversation

olehermanse
Copy link
Member

  • Fixed bug with adding URL at specific commit
  • Added test for adding a URL at specific commit
  • Fixed problems with adding/cloning the same URL and commit twice

Signed-off-by: Ole Herman Schumacher Elgesem <[email protected]>
@olehermanse olehermanse force-pushed the add_bugs branch 2 times, most recently from 34fba82 to cfc1d4c Compare February 2, 2022 13:52
Signed-off-by: Ole Herman Schumacher Elgesem <[email protected]>
If the repo is already cloned, git clone would fail.

Signed-off-by: Ole Herman Schumacher Elgesem <[email protected]>
Copy link
Contributor

@vpodzime vpodzime left a comment

Choose a reason for hiding this comment

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

Looks good to me otherwise.

Signed-off-by: Ole Herman Schumacher Elgesem <[email protected]>
Copy link
Contributor

@vpodzime vpodzime left a comment

Choose a reason for hiding this comment

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

ACK

@olehermanse olehermanse merged commit d59ed18 into cfengine:master Feb 2, 2022
@olehermanse olehermanse deleted the add_bugs branch February 2, 2022 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants