Skip to content
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

Allow [package] on cram stanza #4100

Merged
merged 7 commits into from
Jan 13, 2021
Merged

Conversation

rgrinberg
Copy link
Member

the [package] field will attach the tests of this cram stanza to the
specified package

Fixes #3792

@rgrinberg rgrinberg requested a review from a user January 12, 2021 07:16
@rgrinberg
Copy link
Member Author

Adhoc solution mentioned in #3792 (comment)

Not great, but the lack of this field really hurts the usability of cram tests.

@ghost
Copy link

ghost commented Jan 12, 2021

Looks good to merge otherwise.

src/dune_rules/cram_rules.ml Outdated Show resolved Hide resolved
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

rgrinberg and others added 7 commits January 13, 2021 06:53
the [package] field will attach the tests of this cram stanza to the
specified package

Signed-off-by: Rudi Grinberg <[email protected]>
The plugin stanza did not specify the stanza name correctly. A [~stanza]
label is added to this argument to avoid mistakes like this in the
future.

Signed-off-by: Rudi Grinberg <[email protected]>
Signed-off-by: Rudi Grinberg <[email protected]>
Signed-off-by: Jeremie Dimino <[email protected]>
@rgrinberg rgrinberg merged commit 20cdd87 into ocaml:master Jan 13, 2021
@rgrinberg rgrinberg deleted the cram-package branch January 13, 2021 17:24
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.

opam file generation and test selection
2 participants