-
Notifications
You must be signed in to change notification settings - Fork 154
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
[#1927] Increase minimum version required for Git #2028
[#1927] Increase minimum version required for Git #2028
Conversation
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.
Great job! I think this is very well done, nice that you updated the relevant documentation as well.
Just one minor nitpick is that you missed out deleting a now unused variable GIT_VERSION_INSUFFICIENT_MESSAGE
, located in java/reposense/ConfigSystemTest.java
. Once that is cleaned up, this will be good to merge.
Thanks for the review. Have made the relevant changes and am ready for review again. |
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.
LGTM, nice job!
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.
Thanks for doing this up. Wanted to request some small changes regarding the specification of 2.23 in UG
docs/ug/cli.md
Outdated
<box type="info" seamless> | ||
|
||
* This flag only works on **git `2.23`** or later. | ||
* If an earlier version of **git** is used, RepoSense can still run but this flag will be ignored. | ||
* If an earlier version of **git** is used, RepoSense can still run but only if this flag is not used. | ||
</box> |
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 believe we can remove this section since we are assuming version 2.23.
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.
Thanks for pointing this out, will make the changes accordingly.
docs/ug/configFiles.md
Outdated
<box type="info" seamless> | ||
The Find Previous Authors option only works on **git `2.23`** or later. | ||
</box> | ||
|
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.
Same comment with this part
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.
LGTM!
The following links are for previewing this pull request:
|
Fixes #1927
Proposed commit message
Other information
Changes:
GitVersion
class.UG
andDG
to reflect the increase in Git version required.Notes:
RepoSense will still run perfectly fine with Git version
2.14 - 2.22
provided that the user does not use the--find-previous-authors
flag in the CLI or have theFind Previous Authors
column inrepo-config.csv
set toyes
. Doing otherwise would result in the program failing.