Uh oh!
There was an error while loading. Please reload this page.
Tighten relational operator more - #52342
Conversation
Ryan Cavanaugh (RyanCavanaugh)
commented
Jan 20, 2023
TypeScript Bot (@typescript-bot) test this |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the diff-based user code test suite (tsserver) on this PR at 5f7d2d9. You can monitor the build here. Update: The results are in! |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the parallelized Definitely Typed test suite on this PR at 5f7d2d9. You can monitor the build here. |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the diff-based user code test suite on this PR at 5f7d2d9. You can monitor the build here. Update: The results are in! |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the extended test suite on this PR at 5f7d2d9. You can monitor the build here. |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the diff-based top-repos suite (tsserver) on this PR at 5f7d2d9. You can monitor the build here. Update: The results are in! |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the diff-based top-repos suite on this PR at 5f7d2d9. You can monitor the build here. Update: The results are in! |
Heya Ryan Cavanaugh (@RyanCavanaugh), I've started to run the perf test suite on this PR at 5f7d2d9. You can monitor the build here. Update: The results are in! |
TypeScript Bot (typescript-bot)
commented
Jan 20, 2023
Ryan Cavanaugh (@RyanCavanaugh) Here are the results of running the user test suite comparing Everything looks good! |
TypeScript Bot (typescript-bot)
commented
Jan 20, 2023
Ryan Cavanaugh (@RyanCavanaugh) Here are the results of running the user test suite comparing Something interesting changed - please have a look. Details
|
TypeScript Bot (typescript-bot)
commented
Jan 20, 2023
Heya Ryan Cavanaugh (@RyanCavanaugh), I've run the RWC suite on this PR - assuming you're on the TS core team, you can view the resulting diff here. |
Ryan Cavanaugh (RyanCavanaugh)
commented
Jan 20, 2023
OK, let's analyze.
exportconstOrd: O.Ord<boolean>={equals: Eq.equals,compare: (first,second)=>(first<second ? -1 : first>second ? 1 : 0)}This seems more straightforwardly written as Webpack's is... subtle: /** * @param {string|number} a first id * @param {string|number} b second id * @returns {-1|0|1} compare result */constcompareIds=(a,b)=>{if(typeofa!==typeofb){returntypeofa<typeofb ? -1 : 1;}if(a<b)return-1;if(a>b)return1;return0;};The reasonable objection at the erroring line ( The other break in webpack, hereseems like it might also be correct-by-construction but the code is quite complex and I'm not really sure what's going on here. RWC turned up several instances of relating |
Ryan Cavanaugh (RyanCavanaugh)
commented
Jan 20, 2023
This allows TLD;R we can't really blame this on implicit coercion, since the implicit coercion that occurs in |
TypeScript Bot (typescript-bot)
commented
Jan 20, 2023
Ryan Cavanaugh (@RyanCavanaugh) Here are the results of running the top-repos suite comparing Something interesting changed - please have a look. Detailspalantir/blueprint
|
TypeScript Bot (typescript-bot)
commented
Jan 20, 2023
Ryan Cavanaugh (@RyanCavanaugh) Here they are:CompilerComparison Report - main..52342
System
Hosts
Scenarios
TSServerComparison Report - main..52342
System
Hosts
Scenarios
StartupComparison Report - main..52342
System
Hosts
Scenarios
Developer Information: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
TypeScript Bot (typescript-bot)
commented
Jan 21, 2023
Ryan Cavanaugh (@RyanCavanaugh) Here are the results of running the top-repos suite comparing Something interesting changed - please have a look. Details
|
Addendum to #52048 per discussion there. Evaluating the real-world code impact before discussing further