Skip to content

Clean up some parts of the compose documentation #1100

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

Merged
merged 5 commits into from
Aug 18, 2025

Conversation

z3ntu
Copy link
Contributor

@z3ntu z3ntu commented Jun 26, 2025

Purpose

Clean up some parts of the compose documentation a bit which was added in #855.

CC @unteem

External contributions

Thank you for your contribution! 🎉

Please ensure the following items are checked before submitting your pull request:

  • I have read and followed the contributing guidelines
  • I have read and agreed to the Code of Conduct
  • I have signed off my commits with git commit --signoff (DCO compliance)
  • I have signed my commits with my SSH or GPG key (git commit -S)
  • My commit messages follow the required format: <gitmoji>(type) title description
  • I have added a changelog entry under ## [Unreleased] section (if noticeable change)
  • I have added corresponding tests for new features or bug fixes (if applicable)

@z3ntu z3ntu changed the title Compose bits Clean up some parts of the compose documentation Jun 26, 2025
@AntoLC AntoLC added the docker label Jun 27, 2025
@AntoLC
Copy link
Collaborator

AntoLC commented Jun 30, 2025

Thank you for your contribution !

Your commits are "Unverified":
image

Did you add your key in https://github.com/settings/keys ?

@z3ntu
Copy link
Contributor Author

z3ntu commented Jun 30, 2025

@AntoLC I think I need to add my new email address to the GPG key (which only has the old one), and also re-import it to GitHub since it shows up as expired. Let me figure out how to do that.

@z3ntu
Copy link
Contributor Author

z3ntu commented Jun 30, 2025

GitHub should also think it's verified now!

@AntoLC AntoLC requested a review from lunika July 3, 2025 12:54
@z3ntu
Copy link
Contributor Author

z3ntu commented Aug 1, 2025

Anything missing here still?

@lunika
Copy link
Member

lunika commented Aug 18, 2025

Hi @z3ntu

A merge commit has been added. I didn't know GitHub allowed us to push on a fork...
So this merge commit should be removed, and then the branch should be rebased with the main one.

Once made, ping me, and I will merge it.

Thanks

z3ntu added 5 commits August 18, 2025 19:02
Follow yamlllint's suggestions and use the correct indentation for
lists.

Signed-off-by: Luca Weiss <[email protected]>
It doesn't look like this is supposed to be there.

Signed-off-by: Luca Weiss <[email protected]>
Mostly give this as an example how a person deploying this knows which
knob to turn.

Signed-off-by: Luca Weiss <[email protected]>
@z3ntu
Copy link
Contributor Author

z3ntu commented Aug 18, 2025

@lunika Done!

@lunika lunika merged commit 0892c05 into suitenumerique:main Aug 18, 2025
18 of 20 checks passed
@lunika
Copy link
Member

lunika commented Aug 18, 2025

Thank you for your contribution.

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

Successfully merging this pull request may close these issues.

3 participants