Skip to content

Comments

Additional s3 documentation#139

Closed
andreparodi wants to merge 3 commits intomozilla:masterfrom
andreparodi:additional_s3_documentation
Closed

Additional s3 documentation#139
andreparodi wants to merge 3 commits intomozilla:masterfrom
andreparodi:additional_s3_documentation

Conversation

@andreparodi
Copy link

minor addition for undocumented s3 env variables needed to use a private s3 compatible object store like minio.

@andreparodi
Copy link
Author

Accidentally included the changes from my other PR. I have reverted them in my latest commit so this PR should just be a couple of extra lines of documentation.

@luser
Copy link
Contributor

luser commented Jun 14, 2017

Rebased and landed as c596150

Thanks for the patch! At some point I think I'd like to break the documentation for each storage backend out to a separate .md file, but documenting undocumented things is always an improvement.

@luser luser closed this Jun 14, 2017
@luser luser assigned luser and andreparodi and unassigned luser Jun 14, 2017
drahnr pushed a commit that referenced this pull request Feb 27, 2023
* Handle signals in dist

* Adds a comment
sylvestre pushed a commit that referenced this pull request Mar 3, 2023
* Handle signals in dist

* Adds a comment
drahnr pushed a commit that referenced this pull request Mar 17, 2023
* Handle signals in dist

* Adds a comment
sylvestre pushed a commit that referenced this pull request Mar 20, 2023
* Handle signals in dist

* Adds a comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants