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 #9318 Avoid Haddock warning with haddock --for-hackage #9319

Merged
merged 2 commits into from
Oct 11, 2023
Merged

Conversation

mpilgrem
Copy link
Collaborator

@mpilgrem mpilgrem commented Oct 8, 2023

The logic of this change is simple: if the existing condition for the use of Haddock's --hyperlinked-source flag is met, don't also include the existing logic for the use of Haddock's --source-* options.

Include the following checklist in your PR:

  • Patches conform to the coding conventions. I have followed the style of the surrounding code.
  • Any changes that could be relevant to users have been recorded in the changelog. I don't think this change is worthy of an entry in the Change Log.
  • The documentation has been updated, if necessary. The underlying feature is undocumented, so there is no documentation to update.
  • Manual QA notes have been included. I've included code comments. I don't think QA notes are required.

Bonus points for added automated tests!

@geekosaur
Copy link
Collaborator

Fourmolu doesn't think you folllowed the style closely enough, apparently.

@mpilgrem
Copy link
Collaborator Author

mpilgrem commented Oct 8, 2023

@geekosaur, sorry on formatting; trying again.

@mpilgrem
Copy link
Collaborator Author

mpilgrem commented Oct 9, 2023

@geekosaur, I think the CI is failing for reasons unconnected with this pull request (based, in part, on this comment: #9320 (comment) which refers to the same problem in terms that suggest it is a familiar problem).

@Mikolaj
Copy link
Member

Mikolaj commented Oct 9, 2023

Oh, yes, the "directory-1.3.8.1 directory-1.3.8.1" CI problem strikes again. Let me restart the jobs and if everything else fails, I will wipe out CI cache.

@Mikolaj
Copy link
Member

Mikolaj commented Oct 9, 2023

It failed in the same way. Let me clear the GHA cache.

@Mikolaj
Copy link
Member

Mikolaj commented Oct 9, 2023

Clearing GHA cache of master branch with https://github.com/haskell/cabal/actions/caches fixed the issue this time.

@mpilgrem: apologies for the inconvenience and thank you for the PR. ;)

@mpilgrem mpilgrem added the merge me Tell Mergify Bot to merge label Oct 9, 2023
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Oct 11, 2023
@mergify mergify bot merged commit 2794dd8 into master Oct 11, 2023
44 checks passed
@mergify mergify bot deleted the fix9318 branch October 11, 2023 23:53
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