Skip to content

docs: Clarify default for ENV FETCHMAIL_PARALLEL#3603

Merged
polarathene merged 6 commits intodocker-mailserver:masterfrom
jsonn:fetchmail-doc
Nov 10, 2023
Merged

docs: Clarify default for ENV FETCHMAIL_PARALLEL#3603
polarathene merged 6 commits intodocker-mailserver:masterfrom
jsonn:fetchmail-doc

Conversation

@jsonn
Copy link
Copy Markdown
Contributor

@jsonn jsonn commented Oct 28, 2023

Make this easier to find when brosing the default environment. Adjust documentation to properly mark the actual default value.

jsonn added 2 commits October 28, 2023 04:33
Make this easier to find when brosing the default environment.
Adjust documentation to properly mark the actual default value.
Copy link
Copy Markdown
Member

@polarathene polarathene left a comment

Choose a reason for hiding this comment

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

Thanks for spotting this and contributing the improvement! ❤️

Comment thread docs/content/config/environment.md Outdated
Comment thread mailserver.env Outdated
@polarathene polarathene added this to the v13.0.0 milestone Oct 28, 2023
@polarathene polarathene changed the title fetchmail: improve FETCHMAIL_PARALLEL documentation docs: Clarify default for ENV FETCHMAIL_PARALLEL Nov 1, 2023
@georglauterbach
Copy link
Copy Markdown
Member

@jsonn please take a look at the review :) I want to have this in v13.0.0 🚀

Comment thread docs/content/config/environment.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: 96a119b

@polarathene
Copy link
Copy Markdown
Member

Thanks for the contribution!

@polarathene polarathene merged commit 0703e01 into docker-mailserver:master Nov 10, 2023
reneploetz pushed a commit to reneploetz/docker-mailserver that referenced this pull request Nov 14, 2023
…#3603)

- Make this easier to find when browsing the example environment file.
- Adjust ENV documentation to properly mark the actual default value.

---------

Co-authored-by: Brennan Kinney <[email protected]>
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.

3 participants