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

Delete TLS.md and update README.md about tests #960

Merged

Conversation

zuiderkwast
Copy link
Contributor

Most of the content of TLS.md has already been copied to README.md in #927.

The description of how to run tests with TLS is moved to tests/README.md.

Descriptions of the additional scripts runtest-cluster, runtest-sentinel and runtest-module are added in tests/README.md.

Links to tests/README.md and src/unit/README.md are added in the top-level README.md along with a brief overview of the make test-* commands.

…t tests

Most of the content of TLS.md has already been copied to README.md in valkey-io#927.

The description of how to run tests with TLS is moved to tests/README.md.

Descriptions of the additional scripts runtest-cluster, runtest-sentinel and
runtest-module are added in tests/README.md.

Links to tests/README.md and src/unit/README.md are added in the top-level
README.md along with a brief overview of the `make test-*` commands.

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
@zuiderkwast zuiderkwast added the documentation Improvements or additions to documentation label Aug 28, 2024
Copy link
Member

@enjoy-binbin enjoy-binbin left a comment

Choose a reason for hiding this comment

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

Did not do the full review (trust you)

@zuiderkwast zuiderkwast merged commit 25dd943 into valkey-io:unstable Aug 28, 2024
43 of 44 checks passed
@zuiderkwast zuiderkwast deleted the delete-duplicated-tls-docs branch August 28, 2024 19:17
madolson pushed a commit that referenced this pull request Sep 2, 2024
Most of the content of TLS.md has already been copied to README.md in
#927.

The description of how to run tests with TLS is moved to
tests/README.md.

Descriptions of the additional scripts runtest-cluster, runtest-sentinel
and runtest-module are added in tests/README.md.

Links to tests/README.md and src/unit/README.md are added in the
top-level README.md along with a brief overview of the `make test-*`
commands.

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
madolson pushed a commit that referenced this pull request Sep 3, 2024
Most of the content of TLS.md has already been copied to README.md in
#927.

The description of how to run tests with TLS is moved to
tests/README.md.

Descriptions of the additional scripts runtest-cluster, runtest-sentinel
and runtest-module are added in tests/README.md.

Links to tests/README.md and src/unit/README.md are added in the
top-level README.md along with a brief overview of the `make test-*`
commands.

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
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
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants