-
Notifications
You must be signed in to change notification settings - Fork 376
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
(test) O3-3517: Run E2E Tests on Push and Pull Requests #821
Conversation
Enhance E2E Testing: Run E2E Tests on Every Commit and PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, @Bcoderx6! I’ve added a few minor comments for you to review.
Also fix the filename: |
Co-authored-by: Jayasanka Weerasinghe <33048395+jayasanka-sack@users.noreply.github.com>
Co-authored-by: Jayasanka Weerasinghe <33048395+jayasanka-sack@users.noreply.github.com>
- uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Jayasanka Weerasinghe <33048395+jayasanka-sack@users.noreply.github.com>
Thank you, @jayasanka-sack! I’ve fixed the issues you mentioned. Please have another look and let me know if there are any more changes needed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for getting this done @Bcoderx6 ! 👏
https://openmrs.atlassian.net/browse/O3-3517
This PR introduces a new GitHub Actions workflow to run end-to-end (E2E) tests on every commit and pull request (PR) to the main branch. The new workflow is based on the existing E2E workflow used for releases but is modified to remove tag checkouts, ensuring that all tests are executed against the head of each monorepo @jayasanka-sack