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

Update package.json for expo@51.0.0 dependency #4

Merged
merged 1 commit into from
May 30, 2024

Conversation

winstromming
Copy link
Contributor

@winstromming winstromming commented May 14, 2024

What do these changes do?

Updates an expo core package dependency.

Why are these changes necessary?

Because @expo/config-plugins@7.2.5 throws an error during the expo-doctor stage and @expo/config-plugins@8.0.4 is the latest compatible version.

✔ Check Expo config for common issues
✔ Check package.json for common issues
✔ Check native tooling versions
✔ Check dependencies for packages that should not be installed directly
✔ Check for common project setup issues
✔ Check for issues with metro config
✔ Check npm/ yarn versions
✔ Check Expo config (app.json/ app.config.js) schema
✖ Check that packages match versions required by installed Expo SDK
✔ Check for legacy global CLI installed locally
✔ Check that native modules do not use incompatible support packages
✔ Check that native modules use compatible support package versions for installed Expo SDK

Detailed check results:

Expected package @expo/config-plugins@~8.0.0
Found invalid:
  @expo/config-plugins@7.2.5

Copy link
Contributor

@oristanovic oristanovic left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for the PR!

@Ulrico972 Ulrico972 merged commit aa07319 into urbanairship:main May 30, 2024
1 check passed
@winstromming winstromming deleted the patch-1 branch June 3, 2024 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants