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

This is a test, please ignore #48

Closed
wants to merge 0 commits into from
Closed

This is a test, please ignore #48

wants to merge 0 commits into from

Conversation

UlisesGascon
Copy link
Member

No description provided.

@UlisesGascon UlisesGascon self-assigned this Dec 7, 2024
Copy link

github-actions bot commented Dec 7, 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) and checked applicability (isCheckApplicableToProjectCategory)?
  • 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 deleted the check-pipeline branch December 7, 2024 18:31
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