Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

use latest NodeJS LTS versions for backend services [MRXN23-294] #1434

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hotzevzl
Copy link
Member

@hotzevzl hotzevzl commented Aug 9, 2023

bump NodeJS to 18.17.0 across api/geoprocessing/webshot

Testing instructions

Everything should work as before 🤷🏼

Feature relevant tickets

https://vizzuality.atlassian.net/browse/MRXN23-294


Checklist before submitting

  • Meaningful commits and code rebased on develop.
  • If this PR adds feature that should be tested for regressions when
    deploying to staging/production, please add brief testing instructions
    to the deploy checklist (docs/deployment-checklist.md)
  • Update CHANGELOG file

@vercel
Copy link

vercel bot commented Aug 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
marxan ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 9, 2023 9:56am

@hotzevzl hotzevzl self-assigned this Aug 11, 2023
@hotzevzl hotzevzl marked this pull request as draft August 11, 2023 10:10
@hotzevzl
Copy link
Member Author

thanks @yulia-bel for reminding me that this is effectively blocked by ZJONSSON/node-unzipper#271

a quick review of alternatives, via the thread above: go-task/go-npm#7

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant