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

RO-2323: oppgradere node til 16.x #544

Closed
wants to merge 1 commit into from
Closed

Conversation

amish1188
Copy link
Contributor

@amish1188 amish1188 commented May 3, 2023

Oppgraderer node 14 til node 16 siden Angular 14 støtter ikke node 18.
Node versjoner høyere enn 14 er strengere mot utdaterte dependencies. Den tvinger oss til å bestemme hvilken dependency må vi installere hvis noe bibliotek krever lavere dependency (f.eks @angular-devkit/architect krever rxjs 6.6.7 mens vi bruker rxjs: "^7.5.6" i vår løsningen), må man bruke 'overrides' property til å gi node en beskjed at vi vil helst bruke 7.5.6 versjonen på alle våre dependencies.
Det var en del problemer med ionic/cli. Tydeligvis var det problemer med os.userInfo metoden som sjekker hvilken bruker kjører appen i høyere node versjoner. Den funka ikke og kastet en feil
image

Fant ikke noe bra løsning på det. Ionic teamet virker til å blåse i det. Kan se at det var snakk om det for tre år siden på ionic team/ionic cli og det står en PR på saken som aldri ble godkjent. Jeg lagde en dummy SHELL variabel som egentlig er et tomt objekt. Det løser problemet for nå.

@github-actions
Copy link

github-actions bot commented May 3, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-544.westeurope.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented May 4, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-544.westeurope.azurestaticapps.net

added overrides

try adding user

removed userinfo

upgrade ionic cli

try adding user with bash

removed password

capactiroe core overrides

updated cli

testing with variable name

env to vars

changes
@github-actions
Copy link

github-actions bot commented May 4, 2023

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-water-056410803-544.westeurope.azurestaticapps.net

@amish1188 amish1188 changed the title upgrade node to 16 RO-2323: oppgradere node til 16.x May 4, 2023
@jorgkv jorgkv closed this Jun 13, 2023
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