TypeScript icon indicating copy to clipboard operation
TypeScript copied to clipboard

[WIP] Disallow merging of `enum` declarations

Open rbuckton opened this issue 10 months ago • 10 comments

This is an experiment to access the impact of disallowing the merging of enum declarations.

rbuckton avatar Feb 24 '25 21:02 rbuckton

Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page.

Also, please make sure @DanielRosenwasser and @RyanCavanaugh are aware of the changes, just as a heads up.

typescript-bot avatar Feb 24 '25 21:02 typescript-bot

@typescript-bot test top400 @typescript-bot user test this

rbuckton avatar Feb 24 '25 21:02 rbuckton

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top400 ✅ Started ✅ Results
user test this ✅ Started ✅ Results

typescript-bot avatar Feb 24 '25 21:02 typescript-bot

@rbuckton Here are the results of running the user tests with tsc comparing main and refs/pull/61262/merge:

Everything looks good!

typescript-bot avatar Feb 24 '25 21:02 typescript-bot

@rbuckton Here are the results of running the top 400 repos with tsc comparing main and refs/pull/61262/merge:

Everything looks good!

typescript-bot avatar Feb 24 '25 23:02 typescript-bot

@typescript-bot test top10.00

rbuckton avatar Mar 20 '25 14:03 rbuckton

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top10.00 ✅ Started

typescript-bot avatar Mar 20 '25 14:03 typescript-bot

@typescript-bot test top1000

rbuckton avatar Mar 20 '25 14:03 rbuckton

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top1000 ✅ Started ✅ Results

typescript-bot avatar Mar 20 '25 14:03 typescript-bot

@rbuckton Here are the results of running the top 400 repos with tsc comparing main and refs/pull/61262/merge:

Everything looks good!

typescript-bot avatar Mar 20 '25 16:03 typescript-bot

@typescript-bot test top999

rbuckton avatar Mar 31 '25 15:03 rbuckton

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top999 ✅ Started 👀 Results

typescript-bot avatar Mar 31 '25 15:03 typescript-bot

@rbuckton Here are the results of running the top 999 repos with tsc comparing main and refs/pull/61262/merge:

Something interesting changed - please have a look.

Details

microsoft/azuredatastudio

16 of 67 projects failed to build with the old tsc and were ignored

extensions/sql-migration/tsconfig.json

extensions/sql-assessment/tsconfig.json

microsoft/vscode

5 of 55 projects failed to build with the old tsc and were ignored

src/tsconfig.tsec.json

typescript-bot avatar Mar 31 '25 20:03 typescript-bot