Skip to content

Fix the issue with not serializing impliedFormat when signature and version of the file are same #48614

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

Merged
merged 2 commits into from
Apr 11, 2022

Conversation

sheetalkamat
Copy link
Member

@sheetalkamat sheetalkamat commented Apr 8, 2022

ad1cebc Ensures that when we want to build with full signatures for incremental correctness check we dont use file version as the signature of the file. (This is more correctness test fix)
a92aaed Actual fix where impliedFormat was not serialized into the buildinfo because version and signature of the file was same

@sheetalkamat sheetalkamat merged commit a1e77ed into main Apr 11, 2022
@sheetalkamat sheetalkamat deleted the impliedFormat branch April 11, 2022 19:39
Jack-Works pushed a commit to Jack-Works/TypeScript that referenced this pull request Apr 22, 2022
…ersion of the file are same (microsoft#48614)

* When checking for incremental correctness always calculate signature

* Fix the issue with not serializing impliedFormat when signature == version of the file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants