Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: specify correct amount of volumes #1171

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Conversation

free-ghz
Copy link
Contributor

@free-ghz free-ghz commented Dec 9, 2024

I'm in the middle of setting this up and was a bit confused by the README.md being a bit ambiguous about volume count. In some places it refers to there being three required volumes, but the subsequent example shows only two.

I found this commit 28bdc6b8 that removes the third volume and changes the language to refer to two volumes.

My conclusion is: there should be two volumes and there's been an editing mistake. This MR is just to fix the forgotten references to three volumes. I did NOT try to understand how anything works or WHY there is only two now.

((I don't think i've made an MR like this before. Should i have made an issue first?))

@buchdag buchdag added the type/docs PR with documentation only changes label Dec 9, 2024
@buchdag
Copy link
Member

buchdag commented Dec 9, 2024

Wow nice catch, thanks @free-ghz 👍

@buchdag buchdag merged commit ea11f22 into nginx-proxy:main Dec 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
type/docs PR with documentation only changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants