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

Add gitHead to package.json in all release workflows #58135

Merged
merged 2 commits into from
Apr 19, 2024

Conversation

jakebailey
Copy link
Member

@jakebailey jakebailey commented Apr 9, 2024

This pulls @Andarist's code from #47932 out and uses it before we publish anything. This is actually reasonable to do everywhere now that our releases come from a checked-in pipeline (whereas before they weren't).

We do not do this in the LKG task because we currently check that in, and it'd be very annoying locally anyway even if we stop checking in the changes. (The checked in LKG on release branches is pointless anyway (beyond referring to directly outside of npm), as the release pipelines have always re-built and published that instead.)

Fixes #36960

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Apr 9, 2024
@jakebailey jakebailey merged commit c50ec79 into microsoft:main Apr 19, 2024
25 checks passed
@jakebailey jakebailey deleted the gitHead branch April 19, 2024 22:54
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.

Provide release commit when publishing to NPM
3 participants