-
Notifications
You must be signed in to change notification settings - Fork 12k
feat(@angular/cli): Allow ability to set budget sizes for your bundles #7458
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
"budgets": { | ||
"description": "Defines the budget allotments for bundles.", | ||
"type": "array", | ||
"items": {"type": "object", |
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.
Split in 2 lines.
"type": "string" | ||
}, | ||
"budget": { | ||
"description": "Threshold to measure against. (in kb)", |
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'd rather have a string or number, and string can contain g
M
or k
or whatever.
@@ -192,6 +192,22 @@ | |||
"description": "Name and corresponding file for environment config.", | |||
"type": "object", | |||
"additionalProperties": true | |||
}, | |||
"budgets": { |
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.
With this scheme, how do I specify the error/warning levels? How do I specify my budget for initial vs total?
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.
It's good to have a proof of concept, but we should design this feature and discuss it around a document tomorrow at the meeting. Do you have such a dcoument?
This PR is the original version of the already merged #8939, closing. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Closes #7139