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

Fix #1, add documentation. #12

Merged
merged 1 commit into from
Nov 19, 2023
Merged

Fix #1, add documentation. #12

merged 1 commit into from
Nov 19, 2023

Conversation

532910
Copy link
Contributor

@532910 532910 commented Nov 17, 2023

No description provided.

Copy link
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

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

I don't think it's enough. There has to be some README with navigation, some options are mandatory while others are not. It's not obvious what's the minimal set of options required to get something running, there are no real-world examples that we are usually expecting from really nice documentation.

Signed-off-by: Sеrgiо Nеmirоwski <sergio@morphbits.ru>
@532910
Copy link
Contributor Author

532910 commented Nov 18, 2023

README updated and links added. I believe no options are mandatory. You should start with zero configuration. Link for example configuration is also added.

@roman-khimov roman-khimov merged commit b8f7397 into master Nov 19, 2023
1 check passed
@roman-khimov roman-khimov deleted the docs branch November 19, 2023 14:39
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