require node >=18

[pamapa]

Checklist

• This PR makes changes to the public API
• I have included links for closing relevant issue numbers

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c2f93d4) 78.16% compared to head (cf9acde) 78.16%.

❗ Current head cf9acde differs from pull request most recent head 3eecb24. Consider uploading reports for the commit 3eecb24 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1231   +/-   ##
=======================================
  Coverage   78.16%   78.16%           
=======================================
  Files          45       45           
  Lines        1731     1731           
  Branches      345      345           
=======================================
  Hits         1353     1353           
  Misses        341      341           
  Partials       37       37           

Flag	Coverage Δ
unittests	78.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[garronej]

Hello Thanks for your work,

Not arguing or anything but would explain the motivation behind this requirement?

It my case it required me to update the Dockerfile that build my SPA.

Best regards,

[pamapa]

Done with a 3.0.0 a major release and node 18 is maintained where 16 is no longer. I plan to keep that dependency now longer though, probably unless a new major release or demanding feature...

[garronej]

Ok, thanks for your answer! However, corect me if I'm mistaking oidc-client-ts is a purely client side library that does not comes with any scripts. So ther's nothing really runing in node there is it?

[garronej]

SSR I guess, my bad