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

Add examples #183

Merged
merged 3 commits into from
Jan 1, 2024
Merged

Add examples #183

merged 3 commits into from
Jan 1, 2024

Conversation

martis42
Copy link
Owner

@martis42 martis42 commented Jan 1, 2024

It is a common best practice to have well documented examples which showcase the usage of a project. Until now we pointed to some of the integration tests for this. They are however a bad fit for this. Their documentation is only the minimum to understand the test cases and often
they test edge cases not relevant for the user.

It is a common best practice to have well documented examples which
showcase the usage of a project. Until now we pointed to some of the
integration tests for this. They are however a bad fit for this. Their
documentation is only the minimum to understand the test cases and often
they test edge cases not relevant for the user.
Many README files in the aspect integration tests simply state the obvious.
We started having a README for each integration test due to the overall
documentation pointing to those tests. Now that we have dedicated examples,
we delete all README files without valuable information.
@martis42 martis42 merged commit 50a084e into main Jan 1, 2024
5 checks passed
@martis42 martis42 deleted the add_examples branch January 1, 2024 12:54
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.

1 participant