Skip to content
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

Various fixes to jest plugin #843

Merged
merged 28 commits into from
Jun 6, 2022
Merged

Various fixes to jest plugin #843

merged 28 commits into from
Jun 6, 2022

Conversation

mweichert
Copy link
Collaborator

  • Fixes testPattern not matching against existing files
  • Fixes buffering of Jest output
  • Ensures that stdout and stderr are used appropriately
  • Fixes issue with colourized output lost from jest

Michael Weichert and others added 27 commits June 1, 2022 07:25
…at the codebase is more consistent and adheres to the project style guide at all times (#695)

* Add dprint and configure it

* Format new files on commit

* Updated package.json

* Minor tweaks. Exclude test data from being formatted

* Fix problem with typechecker not passing. Removing dependency that isn't used any more

* undo formatting on one file

Co-authored-by: Michael Weichert <[email protected]>
Co-authored-by: Alireza Haghshenas <[email protected]>
* exclude folders from dprint formatting, remove eslintrc

* exclude folders from dprint formatting, remove eslintrc
Ran
> @taqueria/[email protected] format
> dprint fmt
…as no initializer and is not definitely assigned in the constructor
…mand, open a file dialog to choose a plugin directory
…ests. Root path is being searched more than once
@mweichert mweichert requested a review from a team June 6, 2022 21:02
@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: e3a100b
Status: ✅  Deploy successful!
Preview URL: https://2ccc7161.taqueria.pages.dev

View logs

Copy link
Contributor

@hu3man hu3man left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mweichert mweichert merged commit 99c0402 into main Jun 6, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jun 6, 2022
@hu3man hu3man deleted the main-clone branch August 5, 2022 18:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants