What was changed:
This PR adds the missing fibers dependency which is a transient dependency of webpack-config-terra through sass-loader@10.
Why it was changed:
Currently, the fibers dependency was installed through @wdio/sync in terra-functional-testing, causing an indirect dependency on that terra-functional-testing regardless of it is needed. Therefore, fibers was explicitly added to webpack-config-terra to decouple that link and to allow for removing of @wdio/sync for a future webdriverio upgrade.
Testing
This change was tested using:
[ ] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[x] Other (please describe below)
[ ] No tests are needed
This was tested by ensuring that the local dev-site works with npm start.
Reviews
In addition to engineering reviews, this PR needs:
[ ] UX review
[ ] Accessibility review
[ ] Functional review
Additional Details
Thank you for contributing to Terra.
@cerner/terra
Summary
What was changed: This PR adds the missing
fibers
dependency which is a transient dependency ofwebpack-config-terra
throughsass-loader@10
.Why it was changed: Currently, the
fibers
dependency was installed through@wdio/sync
interra-functional-testing
, causing an indirect dependency on thatterra-functional-testing
regardless of it is needed. Therefore,fibers
was explicitly added towebpack-config-terra
to decouple that link and to allow for removing of@wdio/sync
for a future webdriverio upgrade.Testing
This change was tested using:
This was tested by ensuring that the local dev-site works with
npm start
.Reviews
In addition to engineering reviews, this PR needs:
Additional Details
Thank you for contributing to Terra. @cerner/terra