Skip to content

Conversation

@Tommypop2
Copy link
Member

@Tommypop2 Tommypop2 commented Jan 19, 2025

Completely refactor the create-solid package to improve the maintainability of the CLI

Using a separate package for each subcommand significantly reduces the risk of features being added in other subcommands (add, for example) impacting the critical create logic.

Removed the more-complete @solid-cli/core CLI package, as well as logic for all subcommands apart from create. These would take a lot more effort to refactor and aren't really necessary for now.

Added tests to create-solid

@Tommypop2 Tommypop2 marked this pull request as draft January 19, 2025 00:39
remove unused `ui` package
This allows for `create-solid` to be imported by another package and used there
@Tommypop2 Tommypop2 marked this pull request as ready for review January 19, 2025 01:07
@Tommypop2 Tommypop2 requested a review from dev-rb January 19, 2025 15:46
This should help catch platform-specific bugs
@Tommypop2 Tommypop2 merged commit 30f8ec8 into solidjs-community:main Jan 19, 2025
1 of 3 checks passed
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.

2 participants