Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: import toolchains to use for copy actions #1355

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

kormide
Copy link
Member

@kormide kormide commented Nov 16, 2023

Support bazel-lib 1.x and 2.x. Please review the first commit only.

Type of change

  • New feature or functionality (change which adds functionality)

Test plan

  • Covered by existing test cases
  • Manual testing; please provide instructions so we can reproduce:
  • Second commit (to be removed) tests against bazel-lib 1.x changes.

@kormide kormide force-pushed the copy-toolchains branch 11 times, most recently from f6d5430 to 2a8a64c Compare November 16, 2023 02:43
@kormide kormide force-pushed the copy-toolchains branch 6 times, most recently from 4eff80a to 1ce8cf2 Compare November 16, 2023 22:17
@kormide kormide force-pushed the copy-toolchains branch 3 times, most recently from c1d9dc9 to 6575ad8 Compare November 16, 2023 23:13
examples/macro/BUILD.bazel Show resolved Hide resolved
@kormide kormide merged commit b42ae33 into aspect-build:main Nov 17, 2023
91 checks passed
@kormide kormide deleted the copy-toolchains branch November 17, 2023 00:14
ralimi added a commit to ralimi/rules_esbuild that referenced this pull request Nov 20, 2023
upport bazel-lib 1.x and 2.x.

This is analogous to aspect-build/rules_js#1355 for rules_js.
alexeagle added a commit to aspect-build/rules_esbuild that referenced this pull request Nov 20, 2023
* feat: import toolchains to use for copy actions

upport bazel-lib 1.x and 2.x.

This is analogous to aspect-build/rules_js#1355 for rules_js.

* chore: use bazel-lib 2.0 locally

* chore: update e2e to work with newer rules_js

* chore: green up

---------

Co-authored-by: Alex Eagle <alex@aspect.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants