Closed Burnett2k closed 3 months ago
Latest commit: 15158c6ca3f35aea22c13ff1101960debb12a4a5
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
spectacle-docs-website | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Mar 12, 2024 2:44pm |
Description
As directed by Cb, we're no longer supporting node16, and will likely not use node matrixes anymore to build / test various node versions.
This pr removes usage of node 16 and matrix, as well as works to clean up some warnings due to using older versions of Github actions such as
setup-node
andactions/cache
etc.Fixes #1322 & #1323
ci.yml
Before updating
ci.yml
(lots of warnings):After updating
ci.yml
(no warnings):create-spectacle.yml
before:
after:
release.yml
before:
after:
release isn't ran until a merge / push is made to master, so I cannot obtain a screenshot of the results without doing some trickery. If needed I can do so, but the changes are essentially the same as the other two files, so seems probably unnecessary.
Type of Change
How Has This Been Tested?
Checklist: (Feel free to delete this section upon completion)
pnpm run check:ci
and all checks pass