-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
[expected-lite] Update port to 0.5.0 #20455
[expected-lite] Update port to 0.5.0 #20455
Conversation
ports/expected-lite/vcpkg.json
Outdated
"version-string": "0.3.0", | ||
"port-version": 1, | ||
"description": "Expected objects in C++11 and later in a single-file header-only library" | ||
"version-semver": "0.5.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.
Can you point to where they say they actually do semver?
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.
(If they don't document that they follow semver rules then this should be just "version"
rather than "version-semver"
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 based it off of the fact that the cmake version check uses the equivalent of SameMajorVersion
compatibility, and the version macros in the header are listed as major.minor.patch, but I can't see it being documented as semver anywhere, so I'll use version
instead.
Thanks @Ryan-rsm-McKenzie ! |
Describe the pull request
What does your PR fix?
Updates expected-lite to 0.5.0
Which triplets are supported/not supported? Have you updated the CI baseline?
Just a port update
Does your PR follow the maintainer guide?
Yes
If you have added/updated a port: Have you run
./vcpkg x-add-version --all
and committed the result?Yes