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

Document multiple public libraries #8364

Merged
merged 1 commit into from
Aug 11, 2022
Merged

Conversation

fgaz
Copy link
Member

@fgaz fgaz commented Aug 11, 2022

About time I write this :D

Closes #6362


Please include the following checklist in your PR:

Please also shortly describe how you tested your change. Bonus points for added tests!

@fgaz fgaz mentioned this pull request Aug 11, 2022
31 tasks
doc/cabal-package.rst Outdated Show resolved Hide resolved
@fgaz fgaz force-pushed the multilibs-docs branch 2 times, most recently from 70a57da to 31124dc Compare August 11, 2022 14:42
Copy link
Member

@jneira jneira left a comment

Choose a reason for hiding this comment

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

perfect, thanks for improving docs

@jneira
Copy link
Member

jneira commented Aug 11, 2022

I suppose it is fine to backport it to 3.8

@Mikolaj
Copy link
Member

Mikolaj commented Aug 11, 2022

@mergify backport 3.8

@mergify
Copy link
Contributor

mergify bot commented Aug 11, 2022

backport 3.8

✅ Backports have been created

@fgaz
Copy link
Member Author

fgaz commented Aug 11, 2022

If you don't mind I'm going to skip the 2 day waiting period, so that 3.8 will have multilibs docs in time for the discourse announcement

@Mikolaj
Copy link
Member

Mikolaj commented Aug 11, 2022

If you don't mind I'm going to skip the 2 day waiting period, so that 3.8 will have multilibs docs in time for the discourse announcement

That's a good idea.

@fgaz
Copy link
Member Author

fgaz commented Aug 11, 2022

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Aug 11, 2022

rebase

✅ Branch has been successfully rebased

@fgaz fgaz merged commit f4270fc into haskell:master Aug 11, 2022
@fgaz fgaz deleted the multilibs-docs branch August 11, 2022 16:47
mergify bot pushed a commit that referenced this pull request Aug 11, 2022
fgaz added a commit that referenced this pull request Aug 11, 2022
Document multiple public libraries (backport #8364)
@Kleidukos
Copy link
Member

@fgaz ah, thank you so much for this PR, I just stumbled across the passive-aggressive sentences that you fix and I was about to raise a ticket :D

@fgaz
Copy link
Member Author

fgaz commented Aug 13, 2022

@Kleidukos you're welcome, but what was passive-aggressive about that?

@Kleidukos
Copy link
Member

@Kleidukos you're welcome, but what was passive-aggressive about that?

The ellipses at the end of each sentence :p

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Visibility field not listed under cabal spec changes.
4 participants