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
feat!: drop abandoned com.github.dcendents:android-maven-gradle-plugin #1252
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1252 +/- ##
=======================================
Coverage 71.08% 71.08%
=======================================
Files 22 22
Lines 1705 1705
=======================================
Hits 1212 1212
Misses 493 493 Continue to review full report at Codecov.
|
a9b4e8e
to
4ac2a2d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added a few comments, most of them related to putting mavenCentral on top of google as mavenCentral has had problems in the past finding google packages, so I think having google on top is preferred.
And the other one is about a typo.
I don't really have much experience publishing to maven, so not sure if the changes are ok.
4ac2a2d
to
8fbf9d3
Compare
c4c3be3
to
050e3a3
Compare
04c1a45
to
fa23fef
Compare
apache#1252) * feat: use maven-publish plugin (mavenCentral) * fix: Apache License url * fix: framework build failure without useAndroidX * fix: add aar build artifact
Motivation, Context & Description
.github.dcendents:android-maven-gradle-plugin
which has been abandonded because maven-publish-plugin is supported by the Android Gradle Plugin 3.6.0 and highergradle build
failure due to missinguseAndroidX
flag. The build is needed to create anaar
release artifact that is bundled with the publish.This PR requires #1251 to be merged in first.
You can see commit: 2338bc6
If you want to see only the changes of this PR.
Testing
npm t
Checklist