Skip to content
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

User guide for environment variables #227

Merged
merged 11 commits into from
May 21, 2024
Merged

User guide for environment variables #227

merged 11 commits into from
May 21, 2024

Conversation

Scienfitz
Copy link
Collaborator

@Scienfitz Scienfitz commented May 2, 2024

A small user guide mentioning userfacing environment variables
based on #226

rendered version on fork: https://scienfitz.github.io/baybe-dev/userguide/envvars.html

@Scienfitz Scienfitz self-assigned this May 2, 2024
@Scienfitz Scienfitz added the documentation Improvements or additions to documentation label May 2, 2024
@AVHopp
Copy link
Collaborator

AVHopp commented May 3, 2024

@Scienfitz will review once the previous PR is finished. Also, would appreciate if you could build the doc on your fork.

@Scienfitz
Copy link
Collaborator Author

@AVHopp rendered version on fork https://scienfitz.github.io/baybe-dev/userguide/envvars.html

docs/userguide/envvars.md Show resolved Hide resolved
docs/userguide/envvars.md Show resolved Hide resolved
docs/userguide/userguide.md Show resolved Hide resolved
@Scienfitz Scienfitz force-pushed the docs/envvars branch 2 times, most recently from daed92c to 9491924 Compare May 16, 2024 11:36
@AdrianSosic AdrianSosic changed the title User guide for envronment variables User guide for environment variables May 16, 2024
Copy link
Collaborator

@AdrianSosic AdrianSosic left a comment

Choose a reason for hiding this comment

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

Hi @Scienfitz, thanks for writing. Here a few suggestions

CHANGELOG.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
docs/userguide/envvars.md Outdated Show resolved Hide resolved
baybe/telemetry.py Outdated Show resolved Hide resolved
@Scienfitz Scienfitz force-pushed the docs/envvars branch 3 times, most recently from b11d298 to 0c7763c Compare May 17, 2024 10:31
@AdrianSosic AdrianSosic merged commit e759ca5 into main May 21, 2024
10 checks passed
@AdrianSosic AdrianSosic deleted the docs/envvars branch May 21, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants