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

Update Test and Build workflow #648

Merged
merged 5 commits into from
Nov 2, 2023
Merged

Update Test and Build workflow #648

merged 5 commits into from
Nov 2, 2023

Conversation

chkuang-g
Copy link
Collaborator

@chkuang-g chkuang-g commented Oct 23, 2023

  • Prevent unintended labels from triggering test workflow. Only tests-requested label would trigger test workflow now.
  • Prevent pull request from triggering build workflows since test
    workflow would build EDM4U.
  • Prevent test workflow from emitting error message just because
    grep command did not get any failed tests.
  • Add info for failed NUnit tests

- Prevent unintended labels from triggering test workflow
- Prevent pull request from triggering build workflows since test
  workflow would build EDM4U.
- Prevent test workflow from emitting error message just because
  `grep` command did not get any failed tests.
@chkuang-g chkuang-g requested a review from a-maurice October 23, 2023 17:33
@chkuang-g chkuang-g added the tests-requested Trigger a quick set of integration tests. label Oct 23, 2023
@chkuang-g chkuang-g removed the request for review from a-maurice October 23, 2023 17:34
@chkuang-g chkuang-g added tests-requested Trigger a quick set of integration tests. and removed tests-requested Trigger a quick set of integration tests. labels Oct 23, 2023
@chkuang-g chkuang-g requested a review from a-maurice October 23, 2023 17:41
@chkuang-g chkuang-g merged commit 7782c4c into master Nov 2, 2023
3 checks passed
@chkuang-g chkuang-g deleted the process/pr-test branch November 2, 2023 23:04
@googlesamples googlesamples locked and limited conversation to collaborators Dec 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tests-requested Trigger a quick set of integration tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants