Dev box: test-harness is CommonJS and should use strip-ansi@^6
#4473
Labels
Bot Services
Required for internal Azure reporting. Do not delete. Do not change color.
bug
Indicates an unexpected problem or an unintended behavior.
customer-reported
Required for internal Azure reporting. Do not delete.
Milestone
Comments
compulim
added
bug
11 tasks
compulim
changed the title
test-harness is CommonJS and should use strip-ansi@^6test-harness is CommonJS and should use strip-ansi@^6
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is it an issue related to Adaptive Cards?
Do you have any screenshots?
No response
What version of Web Chat are you using?
Unrelated
Which distribution are you using Web Chat from?
Unrelated
Which hosting environment does this issue primarily affect?
Others or unrelated
Which browsers and platforms do the issue happened?
No response
Which area does this issue affect?
Others/unrelated
Is this an accessibility issue?
Please describe the bug
In #4392, we bumped to
strip-ansi@7.0.1.However,
strip-ansi@^7is ESM only. We should usestrip-ansi@^6until we ported our code to ESM.Do you see any errors in console log?
No response
How to reproduce the issue?
npm run buildunder/packages/test/harness/What is the expected and actual behavior?
Expected:
HTML tests should run properly.
Actual:
HTML tests fail because
test-harnessis not building properly.Adaptive Card JSON
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: