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

ci: setup GitHub actions #4514

Merged
merged 3 commits into from
Sep 8, 2024
Merged

ci: setup GitHub actions #4514

merged 3 commits into from
Sep 8, 2024

Conversation

timdeschryver
Copy link
Member

@timdeschryver timdeschryver commented Aug 31, 2024

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

First step of #4507

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

First step to make sure everything is set up properly :)
This will run CircleCI and GitHub Actions side by side.

Some stats:

Copy link

netlify bot commented Aug 31, 2024

Deploy Preview for ngrx-io canceled.

Name Link
🔨 Latest commit d7611ce
🔍 Latest deploy log https://app.netlify.com/sites/ngrx-io/deploys/66db478ce99e16000808e3c6

@timdeschryver timdeschryver force-pushed the ci/github-ci-affected branch 2 times, most recently from b7d45cd to 0b03e19 Compare August 31, 2024 19:51
@timdeschryver timdeschryver force-pushed the ci/github-ci-affected branch 2 times, most recently from be7eea2 to 8b42646 Compare August 31, 2024 20:19
@@ -0,0 +1,109 @@
name: main
Copy link
Member Author

@timdeschryver timdeschryver Sep 6, 2024

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 :

https://github.com/ngrx/platform/actions/runs/10742124522

@@ -25,14 +25,8 @@ module.exports = function (config) {
colors: true,
logLevel: config.LOG_INFO,
autoWatch: true,
browsers: ['CustomChrome'],
browsers: ['ChromeHeadless'],
Copy link
Member Author

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)

@brandonroberts
Copy link
Member

Thanks Tim!

Copy link
Contributor

@rainerhahnekamp rainerhahnekamp left a 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?

@timdeschryver
Copy link
Member Author

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.

Copy link
Contributor

@rainerhahnekamp rainerhahnekamp left a 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. 👍

@brandonroberts brandonroberts merged commit 21fe84b into main Sep 8, 2024
11 checks passed
@brandonroberts brandonroberts deleted the ci/github-ci-affected branch September 8, 2024 20:47
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.

3 participants