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
Use default_tsconfig on ts_repositories #20964
Conversation
You can preview 0d0069b at https://pr20964-0d0069b.ngbuilds.io/. |
0d0069b
to
a13ce13
Compare
You can preview a13ce13 at https://pr20964-a13ce13.ngbuilds.io/. |
a13ce13
to
120962f
Compare
You can preview 120962f at https://pr20964-120962f.ngbuilds.io/. |
45ef942
to
34b886a
Compare
You can preview 45ef942 at https://pr20964-45ef942.ngbuilds.io/. |
You can preview 34b886a at https://pr20964-34b886a.ngbuilds.io/. |
WORKSPACE
Outdated
|
||
ts_repositories() | ||
ts_setup_workspace( | ||
default_tsconfig = "@angular_src//packages:tsconfig-build.json" |
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.
nice!
@alexeagle CI is not happy, but that looks like a flake. Can you confirm? |
c8133e4
to
5898c98
Compare
You can preview c8133e4 at https://pr20964-c8133e4.ngbuilds.io/. |
You can preview 5898c98 at https://pr20964-5898c98.ngbuilds.io/. |
5898c98
to
37e1871
Compare
You can preview 37e1871 at https://pr20964-37e1871.ngbuilds.io/. |
37e1871
to
761eb0e
Compare
This helps ensure we use the same tsconfig.json file for all compilations. Next steps are to make it the same tsconfig.json file used by the editor
761eb0e
to
9a817ac
Compare
You can preview 761eb0e at https://pr20964-761eb0e.ngbuilds.io/. |
You can preview 9a817ac at https://pr20964-9a817ac.ngbuilds.io/. |
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.
reapproving to make pullapprove happy.
This helps ensure we use the same tsconfig.json file for all compilations. Next steps are to make it the same tsconfig.json file used by the editor PR Close #20964
This helps ensure we use the same tsconfig.json file for all compilations. Next steps are to make it the same tsconfig.json file used by the editor PR Close angular#20964
This helps ensure we use the same tsconfig.json file for all compilations. Next steps are to make it the same tsconfig.json file used by the editor PR Close angular#20964
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. |
This prevents us repeating the tsconfig attribute everywhere, and possibly having different tsconfig for different targets