mauriciovigolo / keycloak-angular

Easy Keycloak setup for Angular applications.
MIT License
730 stars 280 forks source link

Add .npmrc for older npm compatibility #487

Closed XhmikosR closed 1 year ago

XhmikosR commented 1 year ago

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

XhmikosR commented 1 year ago

@mauriciovigolo this should be merged before #458 so that we keep npm 8.x compatibility.

If you want to go with npm 9.x which is what is shipping with Node.js >= 18, we can drop this PR and I'll update #458 accordingly :)

mauriciovigolo commented 1 year ago

Hey @XhmikosR, I would prefer to keep the compatibility with newer versions of npm, so there is no need the add the file. Is there something else you see that I missing? Thanks! :)

XhmikosR commented 1 year ago

All good then. I'll rebase the deps PR later!