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: support angular 17 #503

Merged
merged 7 commits into from
Nov 30, 2023
Merged

feat: support angular 17 #503

merged 7 commits into from
Nov 30, 2023

Conversation

liamdebeasi
Copy link
Contributor

@liamdebeasi liamdebeasi commented Nov 29, 2023

Updates dependencies to support version 17 of Angular

BREAKING CHANGE: The minimum version of Angular required is now 17. Please updates your apps to use
the latest release of Angular.

resolves #502

@@ -13,7 +13,7 @@
"skipLibCheck": true,
"strict": true,
"target": "es2018",
"lib": ["es2018"],
"lib": ["es2018", "dom"],
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without this, I get the following error on build:

util/ast-util.ts:374:5 - error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.

374     console.error('No app module found. Please add your new class to your component.');
        ~~~~~~~

@liamdebeasi liamdebeasi changed the title Ng17 feat: support angular 17 Nov 29, 2023
@liamdebeasi liamdebeasi marked this pull request as ready for review November 29, 2023 15:20
@liamdebeasi liamdebeasi merged commit 26f0cd9 into main Nov 30, 2023
2 checks passed
@liamdebeasi liamdebeasi deleted the ng17 branch November 30, 2023 18:59
liamdebeasi added a commit that referenced this pull request Jan 9, 2024
liamdebeasi added a commit that referenced this pull request Jan 9, 2024
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.

bug: angular toolkit should work in angular 17
2 participants