Skip to content

Conversation

@jbedard
Copy link
Member

@jbedard jbedard commented Jun 20, 2025

Fix #108
Close #109

Changes are visible to end-users: yes

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

Fix config file not being applied

Test plan

  • Covered by existing test cases

@aspect-workflows
Copy link

aspect-workflows bot commented Jun 20, 2025

Test

1 test target passed

Targets
//examples/minify:minified_diff_test [k8-fastbuild] 27ms

Total test execution time was 27ms. 10 tests (90.9%) were fully cached saving 855ms.


Test

e2e/smoke

1 test target passed

Targets
//:minified_diff_test [k8-fastbuild]                37ms

Buildifier      Format

@jbedard jbedard requested a review from thesayyn June 20, 2025 20:07
@jbedard jbedard enabled auto-merge (squash) June 20, 2025 20:14
@jbedard jbedard merged commit f11636e into main Jun 20, 2025
10 checks passed
@jbedard jbedard deleted the config-cli branch June 20, 2025 20:20
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.

[Bug]: terser.config_file seems to be a no-op

4 participants