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

gha: update to latest actions #235

Merged
merged 5 commits into from
Dec 11, 2024
Merged

Conversation

andrewhsu
Copy link
Member

@andrewhsu andrewhsu commented Dec 10, 2024

jira: DEVPROD-2488

fixed yaml indent syntax, simplified workflow, and updated to latest github actions

Copy link

netlify bot commented Dec 10, 2024

Deploy Preview for docs-ui ready!

Name Link
🔨 Latest commit 8b62e57
🔍 Latest deploy log https://app.netlify.com/sites/docs-ui/deploys/67589f35264f230008d23a1a
😎 Deploy Preview https://deploy-preview-235--docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andrewhsu andrewhsu requested a review from ivotron December 10, 2024 20:06
@andrewhsu andrewhsu changed the title gha: update gha to latest actions gha: update to latest actions Dec 10, 2024
jobs:
build:
runs-on: ubuntu-latest

runs-on: ubuntu-22.04
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious why we set this to a specific version

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably for stability. But we should match whatever netlify uses. Do we know what they use?

Copy link
Member Author

@andrewhsu andrewhsu Dec 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

conservatively locking to ubuntu-22.04 to keep consistent behavior after ubuntu-latest is updated in coming change: actions/runner-images#10636

related to jira DEVPROD-2450

npm i
gulp bundle
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The publish release workflow doesn’t use this so I wonder if it’s even needed

@andrewhsu andrewhsu merged commit ec889f3 into main Dec 11, 2024
6 checks passed
@andrewhsu andrewhsu deleted the DEVPROD-2488-udpate-actions-cache branch December 11, 2024 21:17
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.

4 participants