-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Fix submodule paths when AppSubUrl is not root #11098
Conversation
Fix go-gitea#11002 Signed-off-by: Andrew Thornton <art27@cantab.net>
Unit test failing (related) |
This comment has been minimized.
This comment has been minimized.
@@ -51,14 +52,10 @@ func getRefURL(refURL, urlPrefix, parentPath string) string { | |||
urlPrefixHostname = prefixURL.Host | |||
} | |||
|
|||
// FIXME: Need to consider branch - which will require changes in modules/git/commit.go:GetSubModules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
o no a fixme 😆 - just jocking
Codecov Report
@@ Coverage Diff @@
## master #11098 +/- ##
==========================================
- Coverage 43.44% 43.24% -0.21%
==========================================
Files 600 601 +1
Lines 85135 85599 +464
==========================================
+ Hits 36988 37015 +27
- Misses 43575 44016 +441
+ Partials 4572 4568 -4
Continue to review full report at Codecov.
|
@@ -69,16 +66,16 @@ func getRefURL(refURL, urlPrefix, parentPath string) string { | |||
|
|||
m := match[0] | |||
refHostname := m[2] | |||
path := m[3] | |||
pth := m[3] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤣
ping lgtm |
Backport go-gitea#11098 Fix go-gitea#11002 Signed-off-by: Andrew Thornton <art27@cantab.net>
Fix submodule paths when AppSubUrl is not root Fix go-gitea#11002 Signed-off-by: Andrew Thornton <art27@cantab.net>
Fix #11002
Signed-off-by: Andrew Thornton art27@cantab.net