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

extended semver2 support in paket.template files #3184

Merged
merged 2 commits into from May 25, 2018

Conversation

Projects
None yet
2 participants
@viktor-svub
Contributor

viktor-svub commented May 4, 2018

Attempt at supporting half-open dependency constraints with minimum from current or locked version, but open to semver-declared compatibility.

Example of intended usage would be My.Package ~> LOCKED:Minor >= LOCKEDVERSION.

  • Numeric syntax is "number of segments used", negative counts non-zero segments from end.
  • Unit-testing is done by extending existing scenarios via TestCase, further may be added.
  • Combining with group-binding syntax is expected and supported, like:

LOCKED-NetCore:Patch -or- LOCKED:Patch-NetCore
LOCKED-NetCore:[3] -or- LOCKED:[3]-NetCore
LOCKED-NetCore:[-1] -or- LOCKED:[-1]-NetCore

As side effect, paket.template dependencies can now support inline and full-line comments.
Locked-version dependencies without group bind are assumed to be "Main" and print warnings when resolved from different group.

@viktor-svub viktor-svub changed the title from WIP semver support in paket.template files to extended semver2 support in paket.template files May 9, 2018

@forki

This comment has been minimized.

Member

forki commented May 23, 2018

do we need to update docs?

@viktor-svub

This comment has been minimized.

Contributor

viktor-svub commented May 24, 2018

I can add docs change, if there are no expected changes in the proposed syntax

@forki

This comment has been minimized.

Member

forki commented May 24, 2018

@viktor-svub

This comment has been minimized.

Contributor

viktor-svub commented May 24, 2018

Docs updated, feel free to do any fixups - English not being my first language results in some creative contraptions at times :)

@forki forki merged commit d3c5fc0 into fsprojects:master May 25, 2018

1 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment