AL-Go icon indicating copy to clipboard operation
AL-Go copied to clipboard

[Feature Request] Skipping test on demand via PR description [skip test]

Open fvet opened this issue 2 years ago • 1 comments

To skip the CICD action, you can include any of below texts in your commit messages [skip ci], [ci skip], [no ci], [skip actions], or [actions skip] https://github.blog/changelog/2021-02-08-github-actions-skip-pull-request-and-push-workflows-with-skip-ci/

In addition, in DevOps we've build a custom pipeline step to check for skip running tests - on demand - when creating a PR. In some - agreed - cases, we do allow developers to commit changes (fixing a typo, a simple one-liner code change, updated tooltips / translations, changed RDLC file, ... maybe even updating AL-GO system files? ... having minimal to no impact on tests) and avoid tests to be ran (to make the pipeline complete faster)

This can be done by the developer by including the [skip test] or [test skip] in the description of the PR.

image

I've seen the doNotRunTests setting, but this seems to be a fix setting. https://github.com/microsoft/AL-Go/blob/142cf588df1ff591c278723422a4f16d3236b5f8/Scenarios/settings.md

Is there any option to allow for 'skip running of tests' on demand / as part of your PR?

fvet avatar Oct 15 '23 15:10 fvet

Maybe provide a similar option 'Run Tests' when manually running the CI/CD action

image

fvet avatar Oct 15 '23 17:10 fvet