Skip to content

perf: add support of css-variables to redwood #15

perf: add support of css-variables to redwood

perf: add support of css-variables to redwood #15

Triggered via pull request August 5, 2024 19:56
Status Failure
Total duration 21s
Artifacts

commitlint.yml

on: pull_request
commitlint  /  commitlint
13s
commitlint / commitlint
Fit to window
Zoom out
Zoom in

Annotations

1 error
commitlint / commitlint
You have commit messages with errors ⧗ input: Fix some test warnings (#1062) * fix: paragon's Hyperlink no longer accepts a 'content' attribute * test: ensure all act() calls are async * test: Removed "async" from "describe" Returning a Promise from "describe" is not supported. * fix: DiscussionsSettings tests Previous commit revealed several issues with these tests * Don't nest userAction.click in act() -- nested act() statements have indeterminent behaviour. * Use getBy* instead of findBy* with userAction to avoid nested act() statements * Use fireEvent.click when the onClick handlers need to be called * Use queryBy* instead of getBy* when using .toBeInTheDocument or waitForElementToBeRemoved queryBy* return null when the element is not found. * fix: typo in data-testid Warning: React does not recognize the `data-testId` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `data-testid` instead. * test: Use useLocation to test route changes --------- Co-authored-by: Yusuf Musleh <[email protected]> ✖ subject may not be empty [subject-empty] ✖ type may not be empty [type-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://open-edx-proposals.readthedocs.io/en/latest/oep-0051-bp-conventional-commits.html