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

Checklists V2 #944

Draft
wants to merge 87 commits into
base: main
Choose a base branch
from
Draft

Checklists V2 #944

wants to merge 87 commits into from

Conversation

erjosito
Copy link
Collaborator

@erjosito erjosito commented Oct 1, 2024

This is a PR to bring the checklists framework to a new version. Some of the highlights of the new format:

  • Recommendations are stored in individual YAML files.
  • Checklists are collections of recommendations, that can be created with recommendation "selectors".
  • v1 checklists can still be generated for compatibility with the existing tooling.

Before the migration there is some work that needs to be done:

  • Migration scripts created, to make sure that we don't lose any data.
  • Tooling around analysis of the recommendation body.
  • Schemas for both the reco and the checklist files defined.
  • Github action to verify the schema (as well as other aspects such as the uniqueness of reco names) in a PR.
  • Github action to tag PRs depending on which checklist is impacted by reco updates, specially for ALZ.
  • Automatic generation of v1 checklists at push time - WIP
  • Update documentation

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