Skip to content

refactor(cli): change create command to fit the new architecture - #222

Open
Exeloo wants to merge 3 commits into
refactor/change-start-cmd-to-new-archifrom
refactor/change-create-cmd-to-new-archi
Open

Exeloo wants to merge 3 commits into
refactor/change-start-cmd-to-new-archifrom
refactor/change-create-cmd-to-new-archi

Conversation

@Exeloo

@Exeloo Exeloo commented Sep 13, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Refactor create command to fit the new architecture.
The command should no longer run at the root of a workspace but in the sub folders (where config.type = client, server or lib)

How do you test this PR?

@github-actions github-actions Bot added documentation Improvements or additions to documentation command:build Related to build command command:generate Related to generate command command:start Related to start command command:dev Related to dev command command:create Related to create command labels Sep 13, 2026
@Exeloo
Exeloo changed the base branch from main to refactor/change-start-cmd-to-new-archi September 13, 2026 22:56
@Exeloo
Exeloo added this pull request to stack #221 September 13, 2026 22:56
@github-actions github-actions Bot removed documentation Improvements or additions to documentation command:build Related to build command command:generate Related to generate command command:start Related to start command command:dev Related to dev command labels Sep 13, 2026
Comment on lines 30 to +33
it("rejects with code 'load-failed' when the file throws while loading", async () => {
const error = await parseConfig(fixture("throwing-config.ts")).catch((e) => e);
expect(error).toMatchObject({ constructor: ConfigParseError, code: "load-failed" });
expect(error.cause).toBeInstanceOf(Error);
expect(error.message).toContain("boom");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what is this test ? and especially the error message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command:create Related to create command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants