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

Fix optional test-suite/benchmark type docs #8344

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

cydparser
Copy link
Collaborator

@cydparser cydparser commented Aug 7, 2022

Addresses documentation changes requested in PR #8115. I've reverted some additional changes that I made, since the original PR made type optional for all cabal-version's, but PR #8138 limited change to versions >= 3.8.

@ulysses4ever
Copy link
Collaborator

@cydparser thank you! Feel free to apply the merge-me label when you're ready (or signal if you don't have the permissions). It will take two more days for Mergify to merge.

@cydparser cydparser added the merge me Tell Mergify Bot to merge label Aug 8, 2022
@andreabedini andreabedini force-pushed the fix-optional-type-docs branch from 2444057 to efd3f4f Compare August 10, 2022 18:37
@Mikolaj
Copy link
Member

Mikolaj commented Aug 11, 2022

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Aug 11, 2022

rebase

✅ Branch has been successfully rebased

@andreabedini andreabedini force-pushed the fix-optional-type-docs branch from efd3f4f to 1821cfd Compare August 11, 2022 10:19
@andreabedini andreabedini force-pushed the fix-optional-type-docs branch from 1821cfd to 556d1e6 Compare August 13, 2022 16:39
@Mikolaj
Copy link
Member

Mikolaj commented Aug 16, 2022

Let me rebase to include a workaround to our CI instability and fix the tests.

@Mikolaj
Copy link
Member

Mikolaj commented Aug 16, 2022

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Aug 16, 2022

rebase

✅ Branch has been successfully rebased

@ulysses4ever ulysses4ever force-pushed the fix-optional-type-docs branch from 556d1e6 to 51b1278 Compare August 16, 2022 22:08
@mergify mergify bot merged commit a97e1f1 into haskell:master Aug 18, 2022
@Mikolaj
Copy link
Member

Mikolaj commented Aug 19, 2022

@mergify backport 3.8

@mergify
Copy link
Contributor

mergify bot commented Aug 19, 2022

backport 3.8

✅ Backports have been created

mergify bot added a commit that referenced this pull request Aug 19, 2022
Fix optional test-suite/benchmark type docs (backport #8344)
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Sep 10, 2022
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.

4 participants