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: include upstream docs in READMEs #792

Closed
wants to merge 1 commit into from

Conversation

rti
Copy link
Contributor

@rti rti commented Oct 11, 2024

Intention of this PR is the discussion about docs locations...

@lorenjohnson
Copy link
Contributor

Is this a good place to mention that I think it is a mistake to include the docker-compose configurations in the READMEs of the individual images? The thought here is that it is a maintenance headache, or liability, as I think they are likely to get out-of-date and cause confusion. I think it would be better if we simply linked to the Deploy configuration as reference in those places in the READMEs, and leave out the docker-compose samples in the image READMEs. Should I make this its own ticket or can we make it part of the scope of what is being done here?

@rti
Copy link
Contributor Author

rti commented Nov 7, 2024

As we started to look a bit more into docs (e.g. #800) I would close this for now trusting that it will come up naturally again at some point soon.

@rti rti closed this Nov 7, 2024
@rti rti deleted the docs-for-upstream-components branch January 27, 2025 11:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants