-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
ci: setup GitHub actions #4514
ci: setup GitHub actions #4514
Conversation
✅ Deploy Preview for ngrx-io canceled.
|
b7d45cd
to
0b03e19
Compare
be7eea2
to
8b42646
Compare
98e32c8
to
d7611ce
Compare
@@ -0,0 +1,109 @@ | |||
name: main |
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.
I enabled this pipeline on PRs to test it out and then disabled it once the build was successful, here's one that's successful :
@@ -25,14 +25,8 @@ module.exports = function (config) { | |||
colors: true, | |||
logLevel: config.LOG_INFO, | |||
autoWatch: true, | |||
browsers: ['CustomChrome'], | |||
browsers: ['ChromeHeadless'], |
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.
Required, otherwise the CI needs to install chromium (which we did in CircleCI via an orb)
Thanks Tim! |
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.
Looks very structured, very clean. 👍 Really nice!
The configuration for Circle CI will be removed in this PR or in another one?
Yea, it's my intention to keep them in parallel for a while. After v19 we can fully migrate and include the release step. |
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.
As said before, well structured and if something doesn't work out, we still have CircleCI as fallback. 👍
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
First step of #4507
What is the new behavior?
Does this PR introduce a breaking change?
Other information
First step to make sure everything is set up properly :)
This will run CircleCI and GitHub Actions side by side.
Some stats:
Circle CI affect build (PR) 12m 28s - https://app.circleci.com/pipelines/github/ngrx/platform/3713/workflows/55e21343-4028-4ae5-8df5-34c3b6ab4ce0
GitHub affected build (PR) 16m 18s - https://github.com/ngrx/platform/actions/runs/10742124526
GitHub affected build (PR) 7m 43s - https://github.com/ngrx/platform/actions/runs/10743257281
CircleCI full build (main) 12m 28s - https://app.circleci.com/pipelines/github/ngrx/platform/3696/workflows/8cce591c-5094-4f1b-97c9-67444c675823
GitHub full build (main) 18m 0s - https://github.com/ngrx/platform/actions/runs/10742124522