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

fix: proper @jest/globals import #530

Merged
merged 1 commit into from Sep 6, 2023

Conversation

jgoz
Copy link
Collaborator

@jgoz jgoz commented Aug 28, 2023

What:

Change import from @jest/globals to use direct import instead of default import.

Why:

Fixes #529.

@jest/globals does not have a default export, so in environments that do not use synthetic default import logic, either import {expect} or import * as globals is needed. I chose the former.

Checklist:

  • Documentation
  • Tests
  • Updated Type Definitions
  • Ready to be merged

@jgoz jgoz requested a review from nickmccurdy August 28, 2023 14:35
@codecov
Copy link

codecov bot commented Aug 28, 2023

Codecov Report

Merging #530 (b3ab3bd) into main (b959a68) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #530   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines          664       664           
  Branches       251       251           
=========================================
  Hits           664       664           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jgoz
Copy link
Collaborator Author

jgoz commented Aug 29, 2023

@nickmccurdy one more fix to the ESM stuff

nickmccurdy

This comment was marked as resolved.

@jgoz

This comment was marked as resolved.

@nickmccurdy

This comment was marked as resolved.

@nickmccurdy

This comment was marked as resolved.

@nickmccurdy nickmccurdy merged commit 5b492ac into testing-library:main Sep 6, 2023
7 checks passed
@github-actions
Copy link

github-actions bot commented Sep 6, 2023

🎉 This PR is included in version 6.1.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants