-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add GitHub Actions workflow file to run CI #1035
Add GitHub Actions workflow file to run CI #1035
Conversation
✅ Deploy Preview for blissful-goodall-fa23f6 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Thank you!
@@ -19,7 +19,7 @@ jobs: | |||
|
|||
- uses: actions/setup-node@v4.0.1 | |||
with: | |||
node-version: 18 | |||
node-version: 10 |
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.
DevDockerfile and Dockerfile currently still uses node:10-alpine. In website, npm run start:local-api
can run with node 14, but failed with node 16. Potentially using node 14 can be an incremental step to update website.
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.
Good idea! I'll merge this as is and make a new PR to update everything to 14 🙏🏼
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.
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.
Awesome! Thank you!
To complete the switch to GitHub Actions across crawler, service, and website, this adds the same GitHub Actions workflow file that's already running in the other two repos on every pull request.
Once this is merged and verified, I'll follow up with removing the Azure Pipelines config.