Skip to content

pkg: use new type instead of string for versions#9060

Merged
rgrinberg merged 1 commit intoocaml:mainfrom
gridbugs:package-version-type
Nov 1, 2023
Merged

pkg: use new type instead of string for versions#9060
rgrinberg merged 1 commit intoocaml:mainfrom
gridbugs:package-version-type

Conversation

@gridbugs
Copy link
Copy Markdown
Collaborator

@gridbugs gridbugs commented Nov 1, 2023

This adds the Package_version.t type which represents the type of a version of a package. This replaces the use of string for this purpose.

This adds the `Package_version.t` type which represents the type of a
version of a package. This replaces the use of `string` for this
purpose.

Signed-off-by: Stephen Sherratt <stephen@sherra.tt>
@Alizter
Copy link
Copy Markdown
Collaborator

Alizter commented Nov 1, 2023

Wouldn't it be better for Package_version to live in Dune_lang since its not just package specific? Similar to Package_name.

@rgrinberg rgrinberg merged commit 8774c76 into ocaml:main Nov 1, 2023
@gridbugs gridbugs deleted the package-version-type branch November 2, 2023 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants