Generate a checklist from all recommendations #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For our internal tech due diligence, we currently have some checklist, but it would be nice to have one that is in sync with this book automatically. We can autogenerate a checklist from the headings.
I accidentally relied on file system ordering, and it's different between CI and my laptop.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds a small script that collects the recommendations from all chapters and puts them together in a markdown checklist.
We can use this checklist internally for tech due diligence (copy it into a GitHub issue, check the boxes where applicable). It’s not yet complete (e.g. a key management chapter is missing) and often it’s not possible to give a clear yes/no answer, but it can be a starting point, and serve as an overview where we can see at a glance “Red flags, many P0/P1 missing” vs. “P0/P1 all satisfied + some green flags”.
Note for reviewers, this repository is intended to be public!