Replies: 2 comments
-
cc @kwonoj @magic-akari @Austaras cc @devongovett I'm sorry to bother you, but I want to hear your opinion as you are a user of swc while being a long-term maintainer of a huge open-source project at the same time. |
Beta Was this translation helpful? Give feedback.
0 replies
-
I filed wooorm/mdxjs-rs#37 to request permissions about |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I want to hear opinions about two issues. Basically, I want to make a choice that benefits the swc project.
Issue 1: Undeprecating options which is deprecated without discussion with external vendors
We deprecated
ts_enum_is_readonly
intentionally, but there was no discussion beforehand. But I was fine with it while reviewing, because I considerts_enum_is_readonly: true
as something similar toisolatedModules: true
. I think all projects should enable it, and that's all of my reasoning.But it turned out that this can make swc less useful for some external vendors. I think it's better to hear opinions about requests like this, in general.
Issue 2: Maintenance burden vs DX of downstream users
I typically choose the latter, and that's why I created a bot that publishes all related crates when a PR is merged. But this time, the maintenance burden is noticeable.
This is about
#[non_exhaustive]
ofswc_ecma_codegen::Config
. I added it because a breaking change ofswc_core
makes me waste lots of time. A breaking change ofswc_ecma_codegen
is a breaking change ofswc_core
.I need to update
swc_core
ofnext-swc
periodically, but if there's a breaking change ofswc_core
, I have to updatemdxjs-rs
andswc-project/plugins
. Typically, I had to wait for a long time to get the swc_core update PR tomdxjs-rs
got merged.But @dsherret requested to bring it back. My personal feeling is that I don't want to do it, at least considering the amount of contributions from the deno team, but I still want to make a choice for the project.
Beta Was this translation helpful? Give feedback.
All reactions