Skip to content

self.run("clone", "--recurse-submodules", repository, str(dest))#1826

Closed
matemax wants to merge 1 commit into
python-poetry:masterfrom
matemax:clone_submodules
Closed

self.run("clone", "--recurse-submodules", repository, str(dest))#1826
matemax wants to merge 1 commit into
python-poetry:masterfrom
matemax:clone_submodules

Conversation

@matemax
Copy link
Copy Markdown
Contributor

@matemax matemax commented Jan 5, 2020

Fixes: #1828
Closes: #611

Pull Request Check List

  • Added tests for changed code.
    I not found how implement test for these case
  • Updated documentation for changed code.
    I not found docs about clone git dependencies.

@matemax matemax mentioned this pull request Jan 5, 2020
3 tasks
@matemax
Copy link
Copy Markdown
Contributor Author

matemax commented Jan 5, 2020

closes #1828

@finswimmer finswimmer added the kind/feature Feature requests/implementations label Jan 5, 2020
@matemax
Copy link
Copy Markdown
Contributor Author

matemax commented Jan 12, 2020

@finswimmer, i don`t know that this PR ready for review or not because i not found tests which checking a correct repo cloning and docs about clone strategy.

@matemax
Copy link
Copy Markdown
Contributor Author

matemax commented Jan 21, 2020

@sdispater tell me pleaze what should i do yet in it PR?

@matemax
Copy link
Copy Markdown
Contributor Author

matemax commented Mar 7, 2020

Is there any news?

Copy link
Copy Markdown
Member

@finswimmer finswimmer left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

Thanks a lot.

@matemax
Copy link
Copy Markdown
Contributor Author

matemax commented Mar 8, 2020

@sdispater, is there any question, suggestions?

@matemax
Copy link
Copy Markdown
Contributor Author

matemax commented Mar 17, 2020

@finswimmer , @sdispater, Sorry guys, this feature is very helpful for me. What should i do for accepting this PR?

@finswimmer
Copy link
Copy Markdown
Member

Hello @matemax,

@sdispater needs to find some time for reviewing and merging it 😃

fin swimmer

@matemax matemax marked this pull request as draft May 15, 2020 09:49
@matemax matemax closed this May 15, 2020
@matemax matemax mentioned this pull request May 15, 2020
2 tasks
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 1, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

kind/feature Feature requests/implementations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failed add dependency with submodule Add key to allow fetching submodules for git repos

2 participants