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 CLI support for checklist #169

Merged
merged 4 commits into from
Dec 24, 2024
Merged

Add CLI support for checklist #169

merged 4 commits into from
Dec 24, 2024

Conversation

UlisesGascon
Copy link
Member

@UlisesGascon UlisesGascon commented Dec 24, 2024

Main Changes

  • Add CLI command checklist (9dd0767)
  • Allow the checks softwareDesignTraining (db96f64) and githubOrgMFA (758de3b) to support scoped projects.

Other Changes

  • Extend store to support checklist operations (a873707)

Context

Screenshots

List available checklists using CLI

Screenshot from 2024-12-24 22-21-37

Run a checklist using CLI with support

Screenshot from 2024-12-24 22-20-55
Screenshot from 2024-12-24 22-21-17

Run a direct command to execute a checklist with a scoped organziation

Screenshot from 2024-12-24 22-20-01
Screenshot from 2024-12-24 22-20-12

Run a direct command to execute a checklist against all the organizations

Screenshot from 2024-12-24 22-20-37

Changelog

Copy link

github-actions bot commented Dec 24, 2024

It looks like you've made changes to the compliance checks. Thanks for your contribution!
Here are some questions to ensure your changes are complete. Please feel free to ignore the ones that are not relevant:

  • Have you run npm run db:generate-schema to ensure the updated schemas are included in your PR?
  • Have you run npm run db:migrate and then npm run db:rollback to confirm that rollbacks are working as expected?
  • Have you updated the compliance check in the compliance_checks table?
  • Have you included a specific validator (src/checks/validators/) for this check with unit tests (__tests__/checks/)?
  • Have you included a specific file in src/checks/complianceChecks with the integration tests (__tests__/checks/)?
  • Have you included severity validation (getSeverityFromPriorityGroup)?
  • Have you included the tasks, alerts, and results in the database tables?
  • Have you tested the check with check run --name {check_code_name} using the seeded database (npm run db:seed)?
  • Have you created a PR in the website with the calculation details?

You can find more information in the contributing guide.

@UlisesGascon UlisesGascon self-assigned this Dec 24, 2024
@UlisesGascon UlisesGascon marked this pull request as ready for review December 24, 2024 21:37
@UlisesGascon UlisesGascon merged commit 2b4d6b4 into main Dec 24, 2024
4 checks passed
@UlisesGascon UlisesGascon deleted the feat/add-checklists branch December 24, 2024 21:38
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.

Enable checklist
1 participant