Skip to content

Conversation

@alt-romes
Copy link
Collaborator

@alt-romes alt-romes commented Dec 19, 2023

Additionally, a Manual QA note for the same ticket:

QA Note

Running cabal test cabal-install --enable-coverage in the root of the cabal project should succeed and generate a coverage report for cabal-install.


@fgaz I've addressed your comment from #9464. I think all the relevant documentation has already been added but correct me if I'm wrong.

Copy link
Collaborator

@jasagredo jasagredo 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.

@alt-romes alt-romes added the merge me Tell Mergify Bot to merge label Dec 21, 2023
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Dec 23, 2023
Additionally, a Manual QA note for the same ticket:

Running `cabal test cabal-install --enable-coverage` in the root of the
cabal project should succeed and generate a coverage report for
`cabal-install`.
@mergify mergify bot merged commit ef8ab97 into haskell:master Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants