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

Scope package to prevent conflicts with other plugins that haven't bother scoping #10

Merged
merged 3 commits into from
Nov 9, 2019

Conversation

kristfal
Copy link
Contributor

@kristfal kristfal commented Nov 8, 2019

com.reactlibrary package name is the default name when initing a new RN plugin. It should be changed to prevent clashes with other packages that have not yet been renamed.

This is not a breaking change.

Closes #8.

Copy link
Owner

@JuanSeBestia JuanSeBestia left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution

@JuanSeBestia JuanSeBestia merged commit bbe6d67 into JuanSeBestia:master Nov 9, 2019
@JuanSeBestia
Copy link
Owner

🎉 This PR is included in version 2.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sa2812
Copy link

sa2812 commented Dec 8, 2019

I don't use this package, but I have literally spent days trying to find a solution for this problem and I came across this PR. Seeing the changed files showed me problem I was having - thanks so much!

JuanSeBestia pushed a commit that referenced this pull request Mar 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

D8: Program type already present: com.reactlibrary.BuildConfig
3 participants