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

chore: use mdcode to avoid bitrot in README #1172

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

alecthomas
Copy link
Collaborator

@alecthomas alecthomas commented Apr 4, 2024

mdcode is a tool that allows code snippets in Markdown files to be tested. This PR uses it to test that the instructions in the README don't bitrot.

We can/should use this to ensure the instructions in our future documentation stay current.

This required changes to how the controller reports active deployments, some minor refactoring, and adding functionality to the controller that allows it to report readiness only once a set of modules are deployed.

@alecthomas alecthomas mentioned this pull request Apr 4, 2024
[mdcode](https://github.com/szkiba/mdcode) is a tool that allows code
snippets in Markdown files to be tested. This PR uses it to test that
the instructions in the README don't bitrot.

We can/should use this to ensure the instructions in our future
documentation stays current.

This required changes to how the controller reports active deployments,
some minor refactoring, and adding functionality to the controller that
allows it to report readiness only once a set of modules are deployed.
@alecthomas
Copy link
Collaborator Author

cc @johnathan-sq

@alecthomas alecthomas merged commit aa1d1e9 into main Apr 4, 2024
11 checks passed
@alecthomas alecthomas deleted the aat/test-readme branch April 4, 2024 06:12
@johnathan-sq
Copy link

johnathan-sq commented Apr 4, 2024

Awesome that's @alecthomas, cool concept! I'll check it out. PR works for me now. Thanks for that! Keen to start looking into it.

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