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

feat: angular 17 support #257

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

Conversation

ert78gb
Copy link

@ert78gb ert78gb commented Dec 10, 2023

Summary:

  • upgrade all dependency to support angular 17
  • bump GitHub actions image from ubuntu-20.04 => ubuntu-22.04
  • bump actions/checkout from v2 to v4
  • bump actions/setup-node from v2 to v4
  • use .nvmrc file to manage node.js version
  • removed Workaround for JSDOM missing transform property. Not needed anymore
  • use buildTarget instead of deprecated browserTarget in angular.json

BREAKING CHANGE: The upgrade to Angular 17 breaks compatibility with Angular 16.

Summary:
- upgrade all dependency to support angular 17
- bump GitHub actions image from ubuntu-20.04 => ubuntu-22.04
- bump actions/checkout from v2 to v4
- bump actions/setup-node from v2 to v4
- use `.nvmrc` file to manage node.js version
- removed `Workaround for JSDOM missing transform property`. Not needed anymore
- use `buildTarget` instead of deprecated `browserTarget` in `angular.json`

BREAKING CHANGE: The upgrade to Angular 17 breaks compatibility with Angular 16.
@ert78gb ert78gb marked this pull request as ready for review December 10, 2023 10:42
@ert78gb
Copy link
Author

ert78gb commented Dec 10, 2023

Hi @dominique-mueller ,

If you will have time please review this PR.

Thank you,
Robi

@matus-olejnik
Copy link

Hi @dominique-mueller ,

Can you check this this PR please?

Thank you.

@DanTurco
Copy link

DanTurco commented Feb 1, 2024

@dominique-mueller any status on accepting this PR?

@fouratachour
Copy link

fouratachour commented Aug 20, 2024

i just add
"overrides": { "angular-notifier": { "@angular/common": ">=16.0.0", "@angular/core": ">=16.0.0" } }
to my package.json file and it work just fine

@tLau22
Copy link

tLau22 commented Oct 15, 2024

Didn't work for me, is the update still coming?

@ert78gb
Copy link
Author

ert78gb commented Oct 15, 2024

I published a version that support angular 17 https://www.npmjs.com/package/@ert78gb/angular-notifier If you need. I haven't advertised it because I hope this repo will keep maintaining. I haven't upgraded my project to angular 18 so my version support only 17 currently.

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.

5 participants