Skip to content

fix(cli): Clone config instead of updating it#106

Closed
CodeMan62 wants to merge 8 commits intoswc-project:mainfrom
CodeMan62:fix/reuse-config
Closed

fix(cli): Clone config instead of updating it#106
CodeMan62 wants to merge 8 commits intoswc-project:mainfrom
CodeMan62:fix/reuse-config

Conversation

@CodeMan62
Copy link

closes #97

@changeset-bot
Copy link

changeset-bot bot commented May 29, 2025

🦋 Changeset detected

Latest commit: 345edc7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@swc/cli Patch

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

@kdy1
Copy link
Member

kdy1 commented May 30, 2025

I rebased to get CI fixed. Let's see how it goes

@CodeMan62
Copy link
Author

i think it is failing

@kdy1 kdy1 self-assigned this May 30, 2025
@kdy1
Copy link
Member

kdy1 commented May 30, 2025

Can you fix it? The CI passes on the main branch.
The only CI failure on main is publish action.

@CodeMan62
Copy link
Author

Can you fix it? The CI passes on the main branch. The only CI failure on main is publish action.

I will try to fix it and create a PR

@kdy1
Copy link
Member

kdy1 commented May 30, 2025

I meant you should fix CI of this PR

@CodeMan62
Copy link
Author

Sorry i misunderstood give me some time let me fix

@CodeMan62
Copy link
Author

@kdy1 the CI fixed for now but i want to point out that when we do commit the type of err get removed that's why CI get failed

@kdy1 kdy1 changed the title fix reuse of some parts of config fix(cli): Clone config instead of updating it Jun 3, 2025
);
}

await initialCompilation(clonedCliOptions, clonedSwcOptions, callbacks);
Copy link
Member

Choose a reason for hiding this comment

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

This is invoking it twice I guess?

Copy link
Author

@CodeMan62 CodeMan62 Jun 3, 2025

Choose a reason for hiding this comment

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

ahh, you're right i think i made a mistake i am fixing

clonedSwcOptions,
callbacks
);
return watcher;
Copy link
Member

Choose a reason for hiding this comment

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

Why did you add this line?

@CodeMan62 CodeMan62 requested a review from kdy1 June 18, 2025 16:20
@thomasmattheussen
Copy link

@kdy1 @CodeMan62 is there any holdup on this one?

@kdy1
Copy link
Member

kdy1 commented Jun 24, 2025

Yeah, the review is not reflected, and there is too much AI slop.

@kdy1 kdy1 closed this Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

@swc/cli seems to reuse config somewhere under the hood

3 participants