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
fix(compiler-cli): resolve resource URLs before loading them under en… #22688
Conversation
9187a76
to
a425ab6
Compare
You can preview 9187a76 at https://pr22688-9187a76.ngbuilds.io/. |
You can preview a425ab6 at https://pr22688-a425ab6.ngbuilds.io/. |
a425ab6
to
4232892
Compare
You can preview 4232892 at https://pr22688-4232892.ngbuilds.io/. |
…ableResourceInlining Also turn on the feature for Bazel ng_module rules
4232892
to
8594afe
Compare
You can preview 8594afe at https://pr22688-8594afe.ngbuilds.io/. |
delete arg.styleUrls; | ||
} | ||
|
||
const styles = arg['styles']; |
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.
Can simplify with || []
4d4e280
to
a58ec98
Compare
You can preview 4d4e280 at https://pr22688-4d4e280.ngbuilds.io/. |
You can preview a58ec98 at https://pr22688-a58ec98.ngbuilds.io/. |
a58ec98
to
833c8a9
Compare
You can preview 833c8a9 at https://pr22688-833c8a9.ngbuilds.io/. |
833c8a9
to
128771e
Compare
You can preview 128771e at https://pr22688-128771e.ngbuilds.io/. |
128771e
to
dabeb84
Compare
You can preview dabeb84 at https://pr22688-dabeb84.ngbuilds.io/. |
return node; | ||
case 'styleUrls': | ||
if (!ts.isArrayLiteralExpression(prop.initializer)) { | ||
throw new Error('styles takes an array argument'); |
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.
styleUrls
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.
done
if (!ts.isStringLiteral(prop.initializer) && | ||
!ts.isNoSubstitutionTemplateLiteral(prop.initializer)) { | ||
throw new Error( | ||
'Can only accept string literal arguments to templateUrl. ' + PRECONDITIONS_TEXT); |
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.
templateUrl
is not an array, so maybe reword this?
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.
done
…leUrls When both are present, the inlined styles are appended to the end of the styles
dabeb84
to
629cd37
Compare
You can preview 629cd37 at https://pr22688-629cd37.ngbuilds.io/. |
…ableResourceInlining (angular#22688) Also turn on the feature for Bazel ng_module rules PR Close angular#22688
…leUrls (angular#22688) When both are present, the inlined styles are appended to the end of the styles PR Close angular#22688
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. |
…ableResourceInlining
Also turn on the feature for Bazel ng_module rules