-
Notifications
You must be signed in to change notification settings - Fork 48
Remove cosmos from @sei-js/create-sei templates
#283
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
Conversation
- removed the cosmos templates - removed unnecessary config in commands - updated docs to remove unnecessary CLI commands
🦋 Changeset detectedLatest commit: 72a4a00 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes Cosmos templates and functionality from the @sei-js/create-sei dApp generator tool, simplifying the CLI interface and focusing solely on EVM-based development.
- Removes all Cosmos/CosmJS templates and their associated files
- Simplifies CLI by removing ecosystem and library selection options
- Updates documentation to reflect the streamlined template options
Reviewed Changes
Copilot reviewed 43 out of 95 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
packages/create-sei/src/main.ts |
Removes CLI options for ecosystem and library selection, simplifies template selection logic |
packages/create-sei/templates/vite-cosmos-template/* |
Complete removal of Vite + CosmJS template files |
packages/create-sei/templates/next-cosmos-template/* |
Complete removal of Next.js + CosmJS template files |
packages/create-sei/templates/vite-wagmi-template/* |
Complete removal of Vite + Wagmi template files |
packages/create-sei/templates/next-wagmi-template/* |
Complete removal of Next.js + Wagmi template files |
packages/create-sei/templates/vite-template/package.json |
Updates package name to remove wagmi reference |
packages/create-sei/templates/next-template/package.json |
Updates package name to remove wagmi reference |
docs/create-sei/*.mdx |
Updates documentation to remove references to Cosmos templates and simplified CLI commands |
.changeset/tiny-mails-press.md |
Adds changeset for this patch release |
Comments suppressed due to low confidence (2)
packages/create-sei/templates/vite-template/package.json:2
- [nitpick] The package name change removes 'wagmi' but this template may still be using Wagmi. Consider verifying that the template name accurately reflects its technology stack.
"name": "template-vite-sei-app",
packages/create-sei/templates/next-template/package.json:2
- [nitpick] The package name change removes 'wagmi' but this template may still be using Wagmi. Consider verifying that the template name accurately reflects its technology stack.
"name": "template-next-sei-app",
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #283 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 70 70
Lines 844 844
Branches 139 139
=========================================
Hits 844 844 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| @@ -0,0 +1,5 @@ | |||
| --- | |||
| "@sei-js/create-sei": patch | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be a minor/major instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it will be when we redo it completely. So basically the next one
This PR removes cosmos from the boilerplate dApp generator.
Changes