Title: Redo package size changes checks in CI · Issue #53153 · microsoft/TypeScript · GitHub
Open Graph Title: Redo package size changes checks in CI · Issue #53153 · microsoft/TypeScript
X Title: Redo package size changes checks in CI · Issue #53153 · microsoft/TypeScript
Description: We used to get this via the LKG task. After #52226, LKG no longer exists. To ensure that we can still get some idea of when we accidentally regress package size too much, I had to add some hacks to the smoke test in CI which builds LKG a...
Open Graph Description: We used to get this via the LKG task. After #52226, LKG no longer exists. To ensure that we can still get some idea of when we accidentally regress package size too much, I had to add some hacks to...
X Description: We used to get this via the LKG task. After #52226, LKG no longer exists. To ensure that we can still get some idea of when we accidentally regress package size too much, I had to add some hacks to...
Opengraph URL: https://github.com/microsoft/TypeScript/issues/53153
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Redo package size changes checks in CI","articleBody":"We used to get this via the LKG task.\r\n\r\nAfter #52226, LKG no longer exists. To ensure that we can still get some idea of when we accidentally regress package size too much, I had to add some hacks to the `smoke` test in CI which builds LKG at main, saves it outside the tree, then switches back to the PR and copies lib back, as though it were there. Then our old package size checks can kick in.\r\n\r\nThis is a hack and just temporary. What we should actually do is introduce a new CI task which checks this explicitly, then eliminate size checks from our LKG task itself.\r\n\r\nThe best thing would be to create some failure threshold in CI (right now, 10%?), that fails the build. But, using GitHub's checks API, we could feasibly also output a markdown report into the UI to look at. (This is where I'd like to stick perf results and other changes in the future.)\r\n\r\nI've looked at the existing actions in the marketplace, and none really do what we want, in that they all reply via a comment, which will be noisy. Not sure what to do about that, besides writing a whole new action from scratch. Maybe that's fine, because there are other actions I'd like to write too (e.g. the errors delta repo could be an action that runs on every PR, as could DT and perf).\r\n\r\nSee also https://github.com/microsoft/TypeScript/pull/53123.","author":{"url":"https://github.com/jakebailey","@type":"Person","name":"jakebailey"},"datePublished":"2023-03-07T23:15:32.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/53153/TypeScript/issues/53153"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:292c7530-8bd1-99f3-2ac2-dfd66e71967e |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | C986:196FB5:830FF2:B41A99:6A61B863 |
| html-safe-nonce | 234a768f27d4c493e579e863583f4acd11ff9f817fb3eee639aefa6f00f3e326 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDOTg2OjE5NkZCNTo4MzBGRjI6QjQxQTk5OjZBNjFCODYzIiwidmlzaXRvcl9pZCI6IjU5NjUzNjc3MzUxODY3MzMxNTUiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 76fe128ecc49cb3cb3618b15b761131350060dc1003d0cd09180d8d45cf3887a |
| hovercard-subject-tag | issue:1614359365 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/microsoft/TypeScript/53153/issue_layout |
| twitter:image | https://opengraph.githubassets.com/a4595638b485bdd0d330d58194fe9006774e802cdebabdf5fd2216cde0ee85e0/microsoft/TypeScript/issues/53153 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/a4595638b485bdd0d330d58194fe9006774e802cdebabdf5fd2216cde0ee85e0/microsoft/TypeScript/issues/53153 |
| og:image:alt | We used to get this via the LKG task. After #52226, LKG no longer exists. To ensure that we can still get some idea of when we accidentally regress package size too much, I had to add some hacks to... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | jakebailey |
| hostname | github.com |
| expected-hostname | github.com |
| None | 6f4633bcf01c1ad14b73fd07dd39ac31d61f3d3c2578ee08ec1792b7b351eeb9 |
| turbo-cache-control | no-preview |
| go-import | github.com/microsoft/TypeScript git https://github.com/microsoft/TypeScript.git |
| octolytics-dimension-user_id | 6154722 |
| octolytics-dimension-user_login | microsoft |
| octolytics-dimension-repository_id | 20929025 |
| octolytics-dimension-repository_nwo | microsoft/TypeScript |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 20929025 |
| octolytics-dimension-repository_network_root_nwo | microsoft/TypeScript |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | ac296ae7f21856f1f92adbad22f870b6fbb4b907 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width