Skip to content
This repository has been archived by the owner on Sep 24, 2024. It is now read-only.

CI CodeBuild is needed also for merge for documentation changes only #80

Open
tkilias opened this issue Jun 8, 2023 · 1 comment
Open
Labels
feature Product feature

Comments

@tkilias
Copy link
Collaborator

tkilias commented Jun 8, 2023

Background

  • Currently, we have the CI CodeBuild as a required check in the BranchProtection for main
  • This means we need to run it successful to merge into main
  • However, this also means we need to run it for documentation changes only
  • The CI CodeBuild is rather slow and expensive and we would like to avoid running for documentation change only, but still keep it in BranchProtection
@tkilias tkilias added the feature Product feature label Jun 8, 2023
@tkilias
Copy link
Collaborator Author

tkilias commented Jun 8, 2023

Possible solutions:

  • have not the CodeBuild it self as mandatory in the BranchProtection, but a Github Workflow if has run for non-documentation changes
  • keep it in the branch protection, but in case of documentation changes run no tests and add a commit message trigger to override this behavior
    • we could also use a skip tests commit message trigger to skip the tests
    • this could be a specific nox session

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature Product feature
Projects
None yet
Development

No branches or pull requests

1 participant