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

Bump vfs and twoslash versions #2524

Merged
merged 3 commits into from
Sep 7, 2022
Merged

Bump vfs and twoslash versions #2524

merged 3 commits into from
Sep 7, 2022

Conversation

andrewbranch
Copy link
Member

To publish the change from #2522

Comment on lines 1 to 15
undecided:
- "@typescript/ata"
- community-meta
- create-typescript-playground-plugin
- documentation
- glossary
- handbook-epub
- "@typescript/playground"
- playground-examples
- "@typescript/playground-handbook"
- "@typescript/playground-worker"
- "@typescript/sandbox"
- tsconfig-reference
- "@typescript/vfs"
- typescriptlang-org
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this? Am I supposed to commit it?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC @orta said you didn't need to do any version stuff; maybe we shouldn't have added the plugin and instead let whatever Orta had run work? #2523 (comment)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think https://github.com/microsoft/TypeScript-Website/blob/v2/.github/workflows/v2-merged-staging.yml actually does this automatically. Maybe the plugin should be removed.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, though I cannot figure out if it handles version bumps, of if that is even a thing that needs to be done.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But to answer the question what is this and should it be committed; this is what yarn 2+ uses to track changes to packages to know which packages need to bump and when. Normally, you would commit it, but I don't think that we actually use yarn to manage versions.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I need to read better. orta said that you just change the version and it does the rest. In which case, I guess you could do yarn version patch --immediate which skips this? (But, I didn't know that this was even the default behavior, I thought deferred was opt-in)

Copy link
Contributor

@orta orta Sep 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My guess is that this is a similar thing to changesets - https://github.com/changesets/changesets - basically in the monorepo you have a declare updates command, it could be some packages have minor some major, then later you do some sort of deploy command which uses this metadata.

IMO you drop it as you're not using it during deploy.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah. That's basically it. But, what do we do in this repo? Just bump the version by hand? Use yarn? Something else? (I have no idea, but I don't think we use yarn deferred changes)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don’t understand the “just update the versions manually” flow. You’re even updating the lockfile by hand? All I was looking for was something to bump every mention of 1.3.4 to 1.3.5.

@andrewbranch
Copy link
Member Author

Deleted the yarn versions artifact; going to try merging.

@andrewbranch andrewbranch merged commit 53fc7d8 into v2 Sep 7, 2022
@jakebailey jakebailey deleted the bump-vfs branch September 7, 2022 18:59
@orta
Copy link
Contributor

orta commented Sep 7, 2022

Ah yeah, I have a script test/verifyPackageVersions which ensures all the versions are correctly synced (which runs on CI I think)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants