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 build-and-test.yml: Enable PR trigger #3320

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

larswmh
Copy link
Member

@larswmh larswmh commented Jan 10, 2023

This PR enables the PR trigger in the build-and-test.yml workflow.

The first manually triggered workflow ran successfully, as seen here.

Once this PR is merged, the build.yml workflow will be disabled. In the next upcoming PR we should then only see this new workflow.

Thanks to @MikeMcC399 for initially providing this workflow and his extensive Cypress efforts in general. This will greatly improve the process of contributing to the project.


Internal Tracking ID: EXPOSUREAPP-14535

This PR removes the commented PR trigger in the [build-and-test.yml](https://github.com/corona-warn-app/cwa-website/blob/master/.github/workflows/build-and-test.yml) workflow. 

The first manually triggered workflow ran successfully, as seen [here](https://github.com/corona-warn-app/cwa-website/actions/runs/3882434456/jobs/6622581219)

Once this PR is merged, the [build.yml](https://github.com/corona-warn-app/cwa-website/blob/master/.github/workflows/build.yml) workflow will be disabled. In the next upcoming PR we should then only see this new workflow.

Thanks to @MikeMcC399 for providing this workflow and his extensive Cypress efforts in general. This will greatly improve the contributors experiences.
@larswmh larswmh merged commit bf6a665 into master Jan 10, 2023
@MikeMcC399
Copy link
Contributor

@larswmh

Thank you for the acknowledgement! 🙂 Also from my side thanks for the teamwork!

@larswmh larswmh deleted the update/build-and-test-workflow branch January 10, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants