-
Notifications
You must be signed in to change notification settings - Fork 99
feat: add support for Angular v11 #118
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
Conversation
👍 |
Thank you very much. Your help is very much appreciated. |
Any estimate on when this will be released? I'd be happy with a |
@Clashsoft I hope to find some time tonight to look at it, I can't guarantee it though. |
@Clashsoft If this is blocker for you, you want to look at the standalone CLI version of angular-cli-ghpages. This version works without any dependencies to Angular: https://www.github.com/angular-schule/angular-cli-ghpages/tree/master/docs%2FREADME_standalone.md |
4a4e47a
to
b251991
Compare
@JohannesHoppe I just made some updates, influenced by the @angular/fire example and from a quick local test it seems to work. However, I'm not sure if I removed any existing use cases or added regressions. Could you take a look at it? |
@beeman I really appreciate your immediate help. I will hurry up to merge this and to test it intensively. But it's Saturday (familyday), so I can only start in the evening! 😉 |
@beeman I just did a quick look. Some minor formating were changed. Did you run prettier? |
ca1977d
to
3f8c3d8
Compare
I totally agree, family time is always a priority!
It looks like prettier is running on pre-commit. Not sure why it keeps on adding the trailing commas. When I edit the files on save they're not there, but after the commit they are 🤔 |
Yeah, prettier has some strange opinions. 😇 But it's still very useful to keep a consistent code-style. I will just run it again, to be sure. |
"husky": "^4.3.0", | ||
"jest": "^26.6.3", | ||
"json-schema-to-typescript": "^9.1.1", | ||
"prettier": "2.1.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is the explanation! There must have been changes in the rules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah Yeah. That might be related!
@JohannesHoppe thanks for the hint, the standalone version worked without problems! |
@beeman @JohannesHoppe any news, when it will be released? |
@splincode Definitely at the beginning of the week. Please use the standalone CLI in the meanwhile. |
Will this fix be dependent on anything likely to be changed in the near future? An experimental API, etc.? I am trying to decide whether I should switch to the standalone version and stick with it, or wait until the release with this fix. |
@VinnyPutty The experimental Builders API has moved to stable with Angular 11. Thus we should be able to use it without breaking changes for the next years, so my recommendation would be:
|
@fmalcher Thank you. I'll do just that. |
? workspace.root | ||
: asWindowsPath(workspace.root); | ||
const buildTarget = { | ||
name: `${context.target.project}:build:production`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i will extend this to
const buildTarget = { name: options.buildTarget || `${context.target.project}:build:production` };
as seen here: angular/angularfire@7e1918a#diff-77ce940ab1bf28b9fd490f028b74d4a439968ace1f095f72216d64f51bed7c59R25
Hello friends. We have a some small challenges here. However, we are working on a solution but it will take a few more days. Please use the standalone CLI until we have a working version. see #120 |
With the release of Angular 11, the
experimental
API to access the workspace has become stable.This package depends on the experimental API and breaks.
This PR is a work-in-progress that updates the latest packages, and fixes
ng-add.ts
. I did not get todeploy/builder.ts
as it does not seem to be a trivial change.I hope to pick it up later this week. If someone else wants to take a shot: here's a start.
As a reference, this and this will be useful.