-
Notifications
You must be signed in to change notification settings - Fork 314
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
Subversion fixes #8572
Subversion fixes #8572
Conversation
c46831a
to
33623c3
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8572 +/- ##
=========================================
Coverage 67.70% 67.70%
Complexity 1007 1007
=========================================
Files 246 246
Lines 7924 7924
Branches 883 883
=========================================
Hits 5365 5365
Misses 2176 2176
Partials 383 383
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
}.map { | ||
revision | ||
it.toString() |
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.
commit msg: Please provide a rationale.
Slightly improve logging in the caller to be able to remove similar logging in some implementations. Signed-off-by: Sebastian Schuberth <[email protected]>
This changes the state of the `updateClient` and only needs to be set once. Signed-off-by: Sebastian Schuberth <[email protected]>
One should not expect checking out the requested revision to always go well, but return the actual revision so the caller can double-check the requested against the actual revision. Signed-off-by: Sebastian Schuberth <[email protected]>
Simplify code to only do a single `svn switch`, and eventually an `svn update` if a path is specified. Signed-off-by: Sebastian Schuberth <[email protected]>
Ensure that always the first algorithm mentioned at [1] is applied. This avoids errors e.g. when updating to a resolved revision of a tag path. Fixes #6160. [1]: https://svnbook.red-bean.com/en/1.7/svn.advanced.pegrevs.html Signed-off-by: Sebastian Schuberth <[email protected]>
Recent changes made the test work. However, it may take up to 20 minutes to complete, so mark it as expensive. Signed-off-by: Sebastian Schuberth <[email protected]>
Please have a look at the individual commit messages for the details.