Skip to content

test: allowJs without outDir or rootDir #880

Draft
jbedard wants to merge 2 commits intoaspect-build:mainfrom
jbedard:allowjs-no-dirmods
Draft

test: allowJs without outDir or rootDir #880
jbedard wants to merge 2 commits intoaspect-build:mainfrom
jbedard:allowjs-no-dirmods

Conversation

@jbedard
Copy link
Copy Markdown
Member

@jbedard jbedard commented Jan 14, 2026


Changes are visible to end-users: yes/no

  • Searched for relevant documentation and updated as needed: yes/no
  • Breaking change (forces users to change their own code or config): yes/no
  • Suggested release notes appear below: yes/no

Test plan

  • Covered by existing test cases
  • New test cases added
  • Manual testing; please provide instructions so we can reproduce:

@aspect-workflows
Copy link
Copy Markdown

aspect-workflows bot commented Jan 14, 2026

Test (Bazel 6.x) (Test)

All tests were cache hits

25 tests (100.0%) were fully cached saving 1s.


Test (Bazel 7.x) (Test)

All tests were cache hits

25 tests (100.0%) were fully cached saving 2s.


Test (Bazel 8.x) (Test)

All tests were cache hits

25 tests (100.0%) were fully cached saving 2s.


Bazel-7

e2e/bzlmodules

All tests were cache hits

1 test (100.0%) was fully cached saving 32ms.


Bazel-8

e2e/bzlmodules

All tests were cache hits

1 test (100.0%) was fully cached saving 30ms.


Bazel-6

e2e/external_dep

All tests were cache hits

6 tests (100.0%) were fully cached saving 262ms.


Bazel-7

e2e/external_dep

All tests were cache hits

6 tests (100.0%) were fully cached saving 289ms.


Bazel-8

e2e/external_dep

All tests were cache hits

6 tests (100.0%) were fully cached saving 291ms.


Bazel-7

e2e/rules_js_v3

All tests were cache hits

1 test (100.0%) was fully cached saving 39ms.


Bazel-8

e2e/rules_js_v3

All tests were cache hits

1 test (100.0%) was fully cached saving 31ms.


Bazel-7

e2e/smoke

All tests were cache hits

2 tests (100.0%) were fully cached saving 107ms.


Bazel-8

e2e/smoke

All tests were cache hits

2 tests (100.0%) were fully cached saving 149ms.


Bazel-6

e2e/test

All tests were cache hits

25 tests (100.0%) were fully cached saving 1s.


Bazel-7

e2e/test

All tests were cache hits

25 tests (100.0%) were fully cached saving 2s.


Bazel-8

e2e/test

All tests were cache hits

25 tests (100.0%) were fully cached saving 2s.


Bazel-6

e2e/worker

All tests were cache hits

6 tests (100.0%) were fully cached saving 209ms.


Bazel-7

e2e/worker

All tests were cache hits

6 tests (100.0%) were fully cached saving 287ms.


Bazel-8

e2e/worker

All tests were cache hits

6 tests (100.0%) were fully cached saving 363ms.


Bazel-8

examples

⚠️ Buildkite build #461 failed.

//transpiler:allow_js_babel_no_outdir_transpile_0 failed to build

allow_js_babel_no_outdir_transpile_0__js_binary failed: error executing Babel command (from target //transpiler:allow_js_babel_no_outdir_transpile_0) bazel-out/k8-opt-exec-ST-d57f47055a04/bin/transpiler/allow_js_babel_no_outdir_transpile_0__js_binary_/allow_js_babel_no_outdir_transpile_0__js_binary d.js --out-file ../../../transpiler/d.js ... (remaining 2 arguments skipped)
 
Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
 
babel:
  d.js does not exist
Failed tests (3)
//transpiler:allow_js_babel_no_outdir_test [k8-fastbuild]🔗
//transpiler:allow_js_babel_outdir_test [k8-fastbuild]🔗
//transpiler:allow_js_outdir_test [k8-fastbuild]🔗

💡 To reproduce the build failures, run

bazel build //transpiler:allow_js_babel_no_outdir_transpile_0

💡 To reproduce the test failures, run

bazel test //transpiler:allow_js_babel_no_outdir_test //transpiler:allow_js_outdir_test //transpiler:allow_js_babel_outdir_test

Buildifier      Format

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 23, 2026

CLA assistant check
All committers have signed the CLA.

@jbedard jbedard force-pushed the allowjs-no-dirmods branch from c97db0b to ed18de0 Compare January 28, 2026 08:50
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.

2 participants