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.

Sign up for GitHub

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

Jump to bottom

chore: use isTestEnvironment instead of __test__ #294

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

chore: use isTestEnvironment instead of __test__ #294

wants to merge 1 commit into from
+10 −10

Conversation

MounirDhahri
Copy link
Member

@MounirDhahri MounirDhahri commented Dec 13, 2024 •
edited by artsyit
Loading

This PR resolves []

Description

This PR updates Palette to use isTestingEnvironment() instead of __TEST__. This is required for the Expo migration.

This will be working for both Eigen and Energy so it shouldn't break anything

📦 Published PR as canary version: 14.0.13--canary.294.2969.0

✨ Test out this PR locally via:

npm install @artsy/[email protected]
# or 
yarn add @artsy/[email protected]

@MounirDhahri MounirDhahri self-assigned this Dec 13, 2024
@artsy-peril artsy-peril bot added the Version: Minor A deploy for new features label Dec 13, 2024
@MounirDhahri MounirDhahri requested a review from damassi December 13, 2024 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damassi damassi Awaiting requested review from damassi

Assignees

@MounirDhahri MounirDhahri

Labels
canary Version: Minor A deploy for new features
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MounirDhahri