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

publish(9.0.3): fix peerDep angular router for v14 #121

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TPullen21
Copy link

Resolves #119

@superjulius
Copy link

@MarkPieszak Can this be reviewed and merged?

@wolfpackt99
Copy link

bump. Would like this merged as well. any issues?

@iLouminate
Copy link

When will this be merged >.<?

@naticaceres
Copy link

@MarkPieszak how can I help get this merged?

@notsaltydev
Copy link

Is it possible to merge this branch soon?

@ghiscoding
Copy link

why do people even use peer dependency range? I mean why not simply use a minimal greater than version instead of a fixed version? I tried upgrading to Angular 15 and it throws an error and I can't work unless I use a --force install. In my own Angular libs, I simply never use any peer deps but if we really want to use peer deps, then why not use something simpler like this:

"@angular/router": ">=10.0.0"

at least this would work with current and future versions instead of being bugged down every time a new Angular version comes out, which is quite often. This would require a lot less maintenance too

@wolfpackt99
Copy link

@TPullen21 can you complete this PR?

@TPullen21
Copy link
Author

@wolfpackt99 I don't have the rights to complete, it's up to @MarkPieszak.

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.

Angular 14 support
7 participants