-
Notifications
You must be signed in to change notification settings - Fork 1
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
CompatHelper: bump compat for HerbGrammar to 0.5, (keep existing compat) #132
CompatHelper: bump compat for HerbGrammar to 0.5, (keep existing compat) #132
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #132 +/- ##
=======================================
Coverage 71.26% 71.26%
=======================================
Files 20 20
Lines 689 689
=======================================
Hits 491 491
Misses 198 198 ☔ View full report in Codecov by Sentry. |
Simplify compat for `Random`
HerbInterpret = "^0.1.3" | ||
HerbSpecification = "0.2" | ||
MLStyle = "^0.4.17" | ||
Random = "1.8.0,1.9.0,1.10.0,1.11.0" | ||
Random = "1.8.0" |
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.
Why not make it ^1.8.0
then? I think Nicolai was using some Random = 1.10
features for a branch (though I am not sure about that anymore).
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.
"=^1.8" is the same as "=1.8" (the caret is the default behavior) and both allow for packages in the range [1.8, 2.0)
.
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.
Look good.
9f4bcea
to
a1f1ca4
Compare
The caret is the default SemVer behavior (https://pkgdocs.julialang.org/v1/compatibility/\#Caret-specifiers) and it seems other projects leave it off.
a1f1ca4
to
1a23938
Compare
This pull request changes the compat entry for the
HerbGrammar
package from^0.4.0
to^0.4.0, 0.5
.This keeps the compat entries for earlier versions.
Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.