Skip to content

Conversation

realtimetodie
Copy link
Contributor

Renames the Bazel target to @aspect_rules_swc//swc:swc_compile

#88 (comment)

Renames the Bazel target to @aspect_rules_swc//swc:swc_compile
@realtimetodie
Copy link
Contributor Author

realtimetodie commented Jan 31, 2023

The Aspect website needs to be updated too

https://docs.aspect.build/rules/aspect_rules_swc/#in-a-custom-rule

@jbedard jbedard merged commit 03b5ce0 into aspect-build:main Jan 31, 2023
@jbedard
Copy link
Member

jbedard commented Jan 31, 2023

Thanks!

@alexeagle
Copy link
Member

This looks wrong, I don't think this target exists. Since we dropped the nodejs entry point, we don't have any *_binary target in our repo, there is only the external one given by ctx.toolchains["@aspect_rules_swc//swc:toolchain_type"].swcinfo.swc_binary

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.

3 participants