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

chore: deleted plugins that now live outside the monorepo #3899

Merged
merged 1 commit into from Nov 28, 2022

Conversation

snorrees
Copy link
Contributor

@snorrees snorrees commented Nov 23, 2022

Description

The following plugins will now live in individual repos, on a studio-v2 branch:

They are all currently on npm as a prerelease 2.36.0-v2-studio.x, with studio-v2 tag.

When this PR is merged the following will be done in each repo:

  • mainbranch (v3 version) will be configured to use latesttag
  • studio-v2 branch will be changed from prerelease to use range: 2.36.x in .releaserc.json
  • both versions will have a patch release which means:
    • v3 version will have latest tag on npm,
    • v2 version will have studio-v2 tag on npm

What was done in the standalone repos

  • Made a branch based on the first commit in the repo
  • Tagged this commit with v2.35.2 (to give semantic-release something to start from)
  • Deleted src
  • Copied over:
    • src
    • package.json
    • README.md
    • sanity.json
    • config.dist.json (if applicable)
  • Fixed up package.json to use sanipack++ and added other dependencies that where not listed but implicitly available in the monorepo thanks to lerna
  • Added scripts, files ect to make things build and pack correctly (js goes into lib, while ts files must go into dts outside lib because sanipack)
  • Updated README with v2 banner & studio-v2 install command
  • other housekeeping things to make it ecosystem workflow compliant
  • Pushed studio-v2 branch with all changes as feat
  • Ran a semantic-release via Github Actions

Tested prerelease version in a v2 studio already configured for these plugins (not in the monorepo).

What to review

Do a double take on this: I have just deleted the packages here (and fixed one instance of code-input in jest-config).
There are various references to code-input in @sanity/cli (workarounds) which I have not touched.

Given that this builds, is this ok as is?

Notes for release

Probably not needed, since nothing (should) have changed to the packages.
The version numbers will no longer be in sync with v2 releases though.

@vercel
Copy link

vercel bot commented Nov 23, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
studio-workshop ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 23, 2022 at 10:55PM (UTC)
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 23, 2022 at 10:55PM (UTC)

* @sanity/code-input
* @sanity/color-input
* @sanity/dashboard
* @sanity/google-maps-input
* @sanity/language-filter
Copy link
Member

@hermanwikner hermanwikner left a comment

Choose a reason for hiding this comment

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

Looks good to me! 👍

@snorrees snorrees merged commit 4f7a083 into next Nov 28, 2022
@snorrees snorrees deleted the remove-v3-plugins branch November 28, 2022 11:51
skogsmaskin pushed a commit that referenced this pull request Dec 8, 2022
* @sanity/code-input
* @sanity/color-input
* @sanity/dashboard
* @sanity/google-maps-input
* @sanity/language-filter
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

2 participants