Skip to content

Add patch-package and postinstall-postinstall#1673

Closed
metehandemir wants to merge 2 commits intoreact-native-community:mainfrom
metehandemir:patch-7
Closed

Add patch-package and postinstall-postinstall#1673
metehandemir wants to merge 2 commits intoreact-native-community:mainfrom
metehandemir:patch-7

Conversation

@metehandemir
Copy link
Copy Markdown
Contributor

📝 Why & how

Added patch-package and postinstall-postinstall.

✅ Checklist

  • Added library to react-native-libraries.json
  • Updated library in react-native-libraries.json
  • Documented in this PR how to use the feature or replicate the bug.
  • Documented in this PR how you fixed or created the feature.

Copy link
Copy Markdown
Member

@Simek Simek left a comment

Choose a reason for hiding this comment

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

Hey @metehandemir, thanks for the contribution, but both of those packages should be installed as dev dependencies, and they do not have strong connection to the React Native ecosystem.

I wonder the root is an invalid dependency categorization, or do we warn for unwanted category (devDependencies) somewhere in our tooling?

@metehandemir
Copy link
Copy Markdown
Contributor Author

My bad, sorry.

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.

2 participants