-
Notifications
You must be signed in to change notification settings - Fork 3.3k
fixed git submodule bug when using relative paths #12156
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
4d1b53a
fixed git submodule bug and added tests
Choudhry18 680ba31
changed the approach now resetting before updating submodules
Choudhry18 4fb6bdd
reset before submodule update, adding test and updating comments
Choudhry18 001048c
removed large file from test
Choudhry18 e561ffd
taking out the test that isn't directly related
Choudhry18 d3ff937
changed the error message for a test
Choudhry18 04cab09
removed a unused_self that is no longer needed
Choudhry18 f76249a
Merge branch 'main' into git_sub_modules
charliermarsh 28f5f4b
Merge branch 'main' into git_sub_modules
charliermarsh 6d31341
Avoid persistence; use our own repo
charliermarsh 8773c4c
Add RustDoc
charliermarsh b730622
Do this recursively
charliermarsh 6c32ee3
Merge branch 'main' into git_sub_modules
woodruffw acfd222
Re-enable tests
woodruffw ea5f03c
Remove duplicate test
charliermarsh 670622c
Merge branch 'main' into git_sub_modules
charliermarsh e0a0eb1
Add a remote relative submodule test
woodruffw File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Re-enable tests
Signed-off-by: William Woodruff <william@yossarian.net>
- Loading branch information
commit acfd2224fc191fdb938c1534096810b5955caee4
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -11,7 +11,6 @@ license = { workspace = true } | |
|
|
||
| [lib] | ||
| doctest = false | ||
| test = false | ||
|
|
||
| [lints] | ||
| workspace = true | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I noticed the
uv-gittests weren't running, thanks to our new astral-bot automation 🙂