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

feat: improve git clone mechanism #1186

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

gabriel-farache
Copy link
Contributor

@gabriel-farache gabriel-farache commented Jul 5, 2024

Improving how git clone is performed:

  • cleaning created local folder after 1st clone fails
  • better log level to clearly inform the use of what's happening

Solves: konveyor/move2kube-api#159

Copy link

github-actions bot commented Jul 5, 2024

Thanks for making a pull request! 😃
One of the maintainers will review and advise on the next steps.

@gabriel-farache gabriel-farache force-pushed the feat/improve_git_clone branch 3 times, most recently from 97aeca1 to 248aa82 Compare July 5, 2024 15:25
@gabriel-farache gabriel-farache changed the title Improve git clone mechanism feat: Improve git clone mechanism Jul 5, 2024
@github-actions github-actions bot added the feat label Jul 5, 2024
@gabriel-farache gabriel-farache changed the title feat: Improve git clone mechanism feat: improve git clone mechanism Jul 5, 2024
Signed-off-by: gabriel-farache <gfarache@redhat.com>
@gabriel-farache gabriel-farache marked this pull request as ready for review July 9, 2024 13:25
@gabriel-farache
Copy link
Contributor Author

On a side note: I find it strange and un-intuitive to force the creation of the source branch if it does not exists when cloning the repo, this should at least be better advertised (hence using warn instead of debug log level) and even, this behaviour should be explicitly enabled by a flag IMO (maybe it is but I could not find it)

@seshapad
Copy link
Member

@gabriel-farache could you please fix the build..seems like some of the libraries are affecting the license compliance.

Copy link
Member

@kmehant kmehant left a comment

Choose a reason for hiding this comment

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

Thank you @gabriel-farache

@seshapad seshapad merged commit cbf1bd1 into konveyor:main Jul 11, 2024
5 of 6 checks passed
@gabriel-farache gabriel-farache deleted the feat/improve_git_clone branch July 11, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants