Skip to content
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 hash of source file text as version for the source file in tsserver #41938

Closed
wants to merge 4 commits into from

Conversation

sheetalkamat
Copy link
Member

This ensures that we can use tsbuildinfo that tsc emits since versions are used for file upto date ness checks.
Eg this will be needed for #41004 to work.

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Dec 12, 2020
@sheetalkamat sheetalkamat added the Experiment A fork with an experimental idea which might not make it into master label Dec 12, 2020
@sheetalkamat sheetalkamat force-pushed the tsserverSourceFileVersioning branch from c605a26 to 57e397f Compare March 2, 2021 22:44
@sheetalkamat sheetalkamat marked this pull request as ready for review March 25, 2021 21:48
@andrewbranch
Copy link
Member

Is this still an experiment? The changes make sense to me.

@sheetalkamat
Copy link
Member Author

It is not but I dont think I can resolve all the merge conflicts for #41004 and get it reviewed before beta so i will hold off on merging this one for next release.

@sheetalkamat sheetalkamat changed the title [Experiment] Use hash of source file text as version for the source file in tsserver Use hash of source file text as version for the source file in tsserver Mar 26, 2021
@sheetalkamat sheetalkamat removed the Experiment A fork with an experimental idea which might not make it into master label Mar 26, 2021
src/server/scriptInfo.ts Show resolved Hide resolved
src/server/scriptInfo.ts Show resolved Hide resolved
@sheetalkamat
Copy link
Member Author

sheetalkamat commented May 10, 2021

Note to self: Need to handle #43474 after the merge from master

@sheetalkamat sheetalkamat force-pushed the tsserverSourceFileVersioning branch from 9923a47 to a400816 Compare May 18, 2021 23:09
@sheetalkamat sheetalkamat added the Experiment A fork with an experimental idea which might not make it into master label Mar 24, 2022
@sheetalkamat sheetalkamat deleted the tsserverSourceFileVersioning branch November 9, 2022 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

5 participants