Skip to content

Report buildInfo size in extendedDiagnostics - #50293

Closed
Sheetal Nandi (sheetalkamat) wants to merge 1 commit into
mainfrom
buildInfoSize
Closed

Report buildInfo size in extendedDiagnostics#50293
Sheetal Nandi (sheetalkamat) wants to merge 1 commit into
mainfrom
buildInfoSize

Conversation

@sheetalkamat

@sheetalkamat Sheetal Nandi (sheetalkamat) commented Aug 12, 2022 •
edited
Loading

Copy link
Copy Markdown
Member

Just draft so i can port it to any branch i need

This reports buildInfo read and wrie size in extended diagnostics

@typescript-bot TypeScript Bot (typescript-bot) added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Aug 12, 2022
sheetalkamat
Sheetal Nandi (sheetalkamat) commented Aug 12, 2022
View reviewed changes
Comment thread src/compiler/tsbuildPublic.ts Outdated
@sheetalkamat

Sheetal Nandi (sheetalkamat) commented Aug 12, 2022

Copy link
Copy Markdown
Member Author

TypeScript Bot (@typescript-bot) pack this

@typescript-bot

TypeScript Bot (typescript-bot) commented Aug 12, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Heya Sheetal Nandi (@sheetalkamat), I've started to run the tarball bundle task on this PR at 7640bc2. You can monitor the build here.

@typescript-bot

TypeScript Bot (typescript-bot) commented Aug 13, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hey Sheetal Nandi (@sheetalkamat), I've packed this into an installable tgz. You can install it for testing by referencing it in your package.json like so:

{
    "devDependencies": {
        "typescript": "https://typescript.visualstudio.com/cf7ac146-d525-443c-b23c-0d58337efebc/_apis/build/builds/131753/artifacts?artifactName=tgz&fileId=33E94ECB6E16BEBBE58AA1F158AAA41BD71392011744FC0F49CD00ACBAF46E9A02&fileName=/typescript-4.9.0-insiders.20220813.tgz"
    }
}

and then running npm install.

There is also a playground for this build and an npm module you can use via "typescript": "npm:@typescript-deploys/pr-build@4.9.0-pr-50293-2".;

DanielRosenwasser
Daniel Rosenwasser (DanielRosenwasser) reviewed Aug 16, 2022
View reviewed changes
Comment thread src/compiler/emitter.ts Outdated
@sheetalkamat

Sheetal Nandi (sheetalkamat) commented Aug 24, 2022

Copy link
Copy Markdown
Member Author

TypeScript Bot (@typescript-bot) pack this

@typescript-bot

TypeScript Bot (typescript-bot) commented Aug 24, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Heya Sheetal Nandi (@sheetalkamat), I've started to run the tarball bundle task on this PR at 56f546b. You can monitor the build here.

@typescript-bot

TypeScript Bot (typescript-bot) commented Aug 24, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hey Sheetal Nandi (@sheetalkamat), I've packed this into an installable tgz. You can install it for testing by referencing it in your package.json like so:

{
    "devDependencies": {
        "typescript": "https://typescript.visualstudio.com/cf7ac146-d525-443c-b23c-0d58337efebc/_apis/build/builds/132629/artifacts?artifactName=tgz&fileId=E5EC25C5B7AF0C522294502EA402B4AEDE63BAEF6509A4A8B5F77A15D29492CA02&fileName=/typescript-4.9.0-insiders.20220824.tgz"
    }
}

and then running npm install.

There is also a playground for this build and an npm module you can use via "typescript": "npm:@typescript-deploys/pr-build@4.9.0-pr-50293-5".;

@sheetalkamat

Sheetal Nandi (sheetalkamat) commented Nov 2, 2022

Copy link
Copy Markdown
Member Author

Ping Andrew Casey (@amcasey) to review this

amcasey
Andrew Casey (amcasey) reviewed Nov 2, 2022
View reviewed changes
Comment thread src/compiler/builder.ts Outdated
Comment thread src/compiler/program.ts Outdated
Comment thread src/compiler/program.ts Outdated
Comment thread src/compiler/emitter.ts Outdated
@sheetalkamat
Sheetal Nandi (sheetalkamat) marked this pull request as draft November 3, 2022 22:28
@sheetalkamat Sheetal Nandi (sheetalkamat) added the Experiment A fork with an experimental idea which might not make it into master label Nov 3, 2022
@sheetalkamat Sheetal Nandi (sheetalkamat) mentioned this pull request Nov 3, 2022
Merged
@sheetalkamat
Sheetal Nandi (sheetalkamat) force-pushed the buildInfoSize branch 5 times, most recently from e07e4e3 to a4963ca Compare November 8, 2022 00:51
@sandersn

Nathan Shively-Sanders (sandersn) commented Mar 31, 2025

Copy link
Copy Markdown
Member

I believe this test branch is unused so I'm closing it. Please re-open it if I'm wrong.

@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@DanielRosenwasser Daniel Rosenwasser (DanielRosenwasser) DanielRosenwasser left review comments
@andrewbranch Andrew Branch (andrewbranch) Awaiting requested review from andrewbranch
Copilot code review Copilot
Copilot will automatically review once the pull request is marked ready for review
+1 more reviewer
@amcasey Andrew Casey (amcasey) amcasey left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@sheetalkamat Sheetal Nandi (sheetalkamat)

Labels

Author: Team Experiment A fork with an experimental idea which might not make it into master For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sheetalkamat @typescript-bot @sandersn @DanielRosenwasser @amcasey