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

Enable CI in GitHub Actions #2451

Merged
merged 2 commits into from
Aug 9, 2022
Merged

Enable CI in GitHub Actions #2451

merged 2 commits into from
Aug 9, 2022

Conversation

theseanything
Copy link
Contributor

@theseanything theseanything commented May 25, 2022

This enables a GitHub Action workflow to run CI using a re-usable workflow template in govuk-infrastructure repo. This is part of the re-platforming work as we migrate away from Jenkins.

@theseanything theseanything marked this pull request as draft May 25, 2022 15:54
@theseanything theseanything force-pushed the deploy-commit-gha branch 2 times, most recently from ad953d3 to 417cf54 Compare August 9, 2022 09:41
@theseanything theseanything marked this pull request as ready for review August 9, 2022 09:41
This prevents the deploy workflow from being run before the application
tests have been completed. This is to ensure that the commit passes
application tests before being deployed.

The 'workflow_run' trigger is being used as these events cannot be
triggered directly by users. This is to prevent non-production team
members from deploying. The 'repository_dispatch' and 'deployment'
trigger were also considered, however those events can be directly
created via API calls hence would give deploy access to non-production
users (as we assign them "write" roles to repos and GitHub currently
doesn't provide more fine grained access controls).
@theseanything theseanything changed the title Enable CI using GitHub Actions Enable CI in GitHub Actions Aug 9, 2022
@theseanything theseanything merged commit 9eea57a into main Aug 9, 2022
@theseanything theseanything deleted the deploy-commit-gha branch August 9, 2022 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants