chore: update workflows

[nickytonline]

Description

Sets the Node.js engine to a minimum of 18.17.0.

What type of PR is this? (check all applicable)

• 🍕 Feature
• 🐛 Bug Fix
• 📝 Documentation Update
• 🎨 Style
• 🧑‍💻 Code Refactor
• 🔥 Performance Improvements
• ✅ Test
• 🤖 Build
• 🔁 CI
• 📦 Chore (Release)
• ⏩ Revert

Related Tickets & Documents

Relates to #72

Mobile & Desktop Screenshots/Recordings

Steps to QA

Added to documentation?

• 📜 README.md
• 📓 docs.opensauced.pizza
• 🍕 dev.to/opensauced
• 📕 storybook
• 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

[github-actions]

Congrats on making your first Pull Request and thanks for taking the time to improve Open Sauced! ❤️! 🎉🍕
Say hello by joining the conversation in our Discord

[nickytonline]

Maybe I'm reading the failure wrong, but doesn't moving to Node.js 18.17.0 fix the engine issue?

Run npm i -g npm@latest
npm ERR! code EBADENGINE
npm ERR! engine Unsupported engine
npm ERR! engine Not compatible with your version of node/npm: npm@10.2.5
npm ERR! notsup Not compatible with your version of node/npm: npm@10.2.5
npm ERR! notsup Required: {"node":"^18.17.0 || >=20.5.0"}
npm ERR! notsup Actual:   {"npm":"8.19.[4](https://github.com/open-sauced/opengraph/actions/runs/7232618888/job/19765768242#step:4:5)","node":"v1[6](https://github.com/open-sauced/opengraph/actions/runs/7232618888/job/19765768242#step:4:7).20.2"}

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2023-12-19T00_11_4[7](https://github.com/open-sauced/opengraph/actions/runs/7232618888/job/19765768242#step:4:8)_407Z-debug-0.log
Error: Process completed with exit code 1.

[bdougie]

Maybe I'm reading the failure wrong, but doesn't moving to Node.js 18.17.0 fix the engine issue?

I would think so, however it seems like something is quite seeing the npm and node change. Does npm run prebuild && npm run build work locally?

[brandonroberts]

Maybe needs fixing here also https://github.com/open-sauced/opengraph/blob/beta/Dockerfile#L20

[brandonroberts]

The workflow files need to be updated to use the open-sauced/hot workflows instead of open-sauced/open-sauced also

https://github.com/open-sauced/app/blob/beta/.github/workflows/compliance.yml#L15
https://github.com/open-sauced/app/blob/beta/.github/workflows/development.yml#L14C1-L14C1

[bdougie]

The workflow files need to be updated to use the open-sauced/hot workflows instead of open-sauced/open-sauced also

https://github.com/open-sauced/app/blob/beta/.github/workflows/compliance.yml#L15 https://github.com/open-sauced/app/blob/beta/.github/workflows/development.yml#L14C1-L14C1

This was helpful. I fix this PR with this - open-sauced/contributor.info@cd6bdc2

[nickytonline]

Not sure why this fails still. All the workflows are updated, the docker file etc. 🫠

[brandonroberts]

The workflows in opensauced/hot are set to use node 18, which would conflict with us trying to use node 20 here

[brandonroberts]

We're using node 18 in the app https://github.com/open-sauced/app/blob/beta/.github/workflows/release.yml#L37

[nickytonline]

Il revert the node 20 upgrade then and just keep the updated workflows. Originally I thought it required Node 20, but it sounds like just updating workflows should be good.

[github-actions]

🎉 This PR is included in version 2.4.1-beta.2 🎉

The release is available on:

• GitHub release
• ghcr.io container (@beta dist-tag)

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 2.5.0 🎉

The release is available on:

• GitHub release
• ghcr.io container (@latest dist-tag)

Your semantic-release bot 📦🚀