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

Recommend package installs & enhance installation & configuration docs #204

Merged
merged 2 commits into from
Oct 17, 2023

Conversation

yhabteab
Copy link
Member

fixes #196

@cla-bot cla-bot bot added the cla/signed label Sep 12, 2023
@yhabteab yhabteab added bug Something isn't working area/configuration labels Sep 12, 2023
@yhabteab yhabteab changed the title Recommend package installs & enchance installation & configuration docs Recommend package installs & enhance installation & configuration docs Sep 12, 2023
@yhabteab yhabteab force-pushed the docs-enhancements branch 3 times, most recently from b977a4c to cab9938 Compare September 12, 2023 13:07
@yhabteab yhabteab requested a review from nilmerg September 12, 2023 13:08
@yhabteab yhabteab self-assigned this Sep 12, 2023
nilmerg
nilmerg previously approved these changes Sep 12, 2023
Copy link
Member

@nilmerg nilmerg left a comment

Choose a reason for hiding this comment

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

Seems fine

@nilmerg nilmerg requested a review from lippserd September 12, 2023 13:48
@nilmerg nilmerg added enhancement New feature or request area/documentation and removed bug Something isn't working area/configuration labels Sep 12, 2023
@nilmerg nilmerg added this to the 1.0.0 milestone Sep 12, 2023
doc/03-Configuration.md Show resolved Hide resolved
doc/03-Configuration.md Outdated Show resolved Hide resolved
nilmerg
nilmerg previously approved these changes Sep 18, 2023
@yhabteab yhabteab force-pushed the docs-enhancements branch 2 times, most recently from d0b2c83 to b5dfbff Compare September 19, 2023 15:11
@yhabteab yhabteab mentioned this pull request Oct 16, 2023
1 task
doc/01-About.md Outdated Show resolved Hide resolved
doc/03-Configuration.md Outdated Show resolved Hide resolved
doc/03-Configuration.md Outdated Show resolved Hide resolved
doc/03-Configuration.md Outdated Show resolved Hide resolved
Copy link
Member

@nilmerg nilmerg left a comment

Choose a reason for hiding this comment

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

I've also tested this locally as if rendered on the website. Looks fine, albeit the notice in 03-Configuration.md about the install without packages will not be hidden there, as the template macros are not evaluated in this context. But that's not a problem here.

@nilmerg nilmerg dismissed lippserd’s stale review October 17, 2023 12:33

requested changes were applied

@nilmerg nilmerg merged commit 6af246f into master Oct 17, 2023
6 checks passed
@nilmerg nilmerg deleted the docs-enhancements branch October 17, 2023 12:34
yhabteab added a commit that referenced this pull request Oct 25, 2023
## Blocked by

- #204
- #209

## TODO

- [x] Rename default branch to `main` (@nilmerg)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect path to MySQL and PostgreSQL database schema files referenced in the installation instructions.
3 participants