Skip to content

chore(expect): use matcher utils via context instead of direct imports #13567

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Nov 6, 2022

Summary

Sort of related to #13362 - if we wanna cleanly pick up colors etc., and expect custom matchers to do so, we need to go through the same mechanisms.

Test plan

Pure refactor, so green CI

@github-actions
Copy link

github-actions bot commented Feb 4, 2023

This PR is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@cpojer
Copy link
Member

cpojer commented May 22, 2025

Probably needs a big rebase/merge.

@SimenB SimenB force-pushed the use-context-in-expect branch from b47d137 to 4c928e4 Compare May 29, 2025 10:10
Copy link

netlify bot commented May 29, 2025

Deploy Preview for jestjs ready!

Name Link
🔨 Latest commit 95c4c8f
🔍 Latest deploy log https://app.netlify.com/projects/jestjs/deploys/6838332646c81000082245db
😎 Deploy Preview https://deploy-preview-13567--jestjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@SimenB SimenB force-pushed the use-context-in-expect branch from 4c928e4 to 95c4c8f Compare May 29, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants