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

Update Dockerfile to node 16 #614

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

AlvaroVega
Copy link
Member

@AlvaroVega AlvaroVega commented Sep 6, 2023

Mimics from telefonicaid/iotagent-json#669

  • Tested

@AlvaroVega AlvaroVega requested a review from fgalan September 6, 2023 10:28
@AlvaroVega AlvaroVega changed the title [WIP] Update Dockerfile to node 16 Update Dockerfile to node 16 Sep 6, 2023
docker/Dockerfile Outdated Show resolved Hide resolved
@fgalan
Copy link
Member

fgalan commented Sep 6, 2023

Similar to PR #599 ?

Co-authored-by: Fermín Galán Márquez <[email protected]>
CHANGES_NEXT_RELEASE Outdated Show resolved Hide resolved
CHANGES_NEXT_RELEASE Outdated Show resolved Hide resolved
@fgalan
Copy link
Member

fgalan commented Sep 6, 2023

Node v16 is not covered by unit test regression at https://github.com/telefonicaid/fiware-sth-comet/blob/master/.github/workflows/ci.yml. Should that be fixed before merging this PR?

@AlvaroVega
Copy link
Member Author

We need to upgrade hapi dependence to a modern one in order to support node 16

@fgalan
Copy link
Member

fgalan commented Nov 3, 2023

We need to upgrade hapi dependence to a modern one in order to support node 16

Being addressed in PR #618

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.

2 participants