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
Remote: new ListContext function #278
Conversation
0c05bd3
to
2ffab4d
Compare
Does anyone know how can I re-run my tests directly rather than change my codes and push again. (Because these failed tests are not related with my changes) |
dd236a3
to
d8220e5
Compare
37bd7e1
to
44978de
Compare
Could anyone pls help review this PR? Maybe @mcuadros, can you help review it pls? Thanks in advance! |
Looks great can you rebase it? |
44978de
to
13be118
Compare
@mcuadros Sure, done! Please take a look again, thanks a lot! |
go.sum
Outdated
@@ -25,24 +25,16 @@ github.com/go-git/go-billy/v5 v5.0.0 h1:7NQHvd9FVid8VL4qVUMm8XifBK+2xCoZ2lSk0agR | |||
github.com/go-git/go-billy/v5 v5.0.0/go.mod h1:pmpqyWchKfYfrkb/UVH4otLvyi/5gJlGI4Hb3ZqZ3W0= | |||
github.com/go-git/go-billy/v5 v5.1.0 h1:4pl5BV4o7ZG/lterP4S6WzJ6xr49Ba5ET9ygheTYahk= | |||
github.com/go-git/go-billy/v5 v5.1.0/go.mod h1:pmpqyWchKfYfrkb/UVH4otLvyi/5gJlGI4Hb3ZqZ3W0= | |||
github.com/go-git/go-git-fixtures/v4 v4.0.1 h1:q+IFMfLx200Q3scvt2hN79JsEzy4AmBTp/pqnefH+Bc= |
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.
please don't update the go.sum
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.
@mcuadros May I know why?
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.
Because the purpose of this PR is to add the ListContext function not update the dependencies.
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 see, done! @mcuadros Please take a look again, thanks!
2867215
to
0cc02f5
Compare
0cc02f5
to
768c024
Compare
It's from this discussion: #246 (comment)
When a user calls
go-git
package to dogit ls-remote
check, we should provide a timeout. Otherwise, this will be stuck in some situation, and cause a controller which is usingList
alway waits again and again.. This is not expected I think.This pull request add a
ListContext
forList
func and passcontext
. For context, I set10s
timeout. If exceed 10s, thenList
func will exit. Maybe10s
is too short, we can discuss a reasonable value about it.Also I add a unit test to test timeout case. Locally, all unit tests passed.