-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Docs build fails with "One or more submodule URLs are not valid." #3857
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
Comments
Confirmed this is a bug. Gitpython reports submodules like:
In the case of relative URLs, git operates not as we'd expect:
So for a parent project with remote And a submodule with URL of |
* Add conditional logic for submodule relative urls * Break down logic a bit more * Add test case for relative url submodule * Broke test case for invalid url submodule Fixes #3857
* Add conditional logic for submodule relative urls * Break down logic a bit more * Add test case for relative url submodule * Broke test case for invalid url submodule Fixes #3857
I forgot to follow up here. This fix went live on monday, so issues with relative submodule urls should be resolved. |
* Fix up some of the logic around repo and submodule URLs * Add conditional logic for submodule relative urls * Break down logic a bit more * Add test case for relative url submodule * Broke test case for invalid url submodule Fixes #3857 * Lint fix
Details
Expected Result
Given our .gitmodules file, the build succeeds.
The last change to our .gitmodules file was 8 days ago (March 19th).
Actual Result
An error:
is displayed and the build fails.
The text was updated successfully, but these errors were encountered: