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

Github packages setup #2

Merged
merged 3 commits into from
Oct 24, 2022
Merged

Github packages setup #2

merged 3 commits into from
Oct 24, 2022

Conversation

bradroid
Copy link

@bradroid bradroid commented Oct 11, 2022

Summary

Adds a script for publishing all the java and android library modules to Github Packages.
I've already ran the script so the packages can be found here.

I've mostly followed this blog to create the script:
https://apiumhub.com/tech-blog-barcelona/publishing-multi-module-android-libraries/#Publishing_multi_module_Android_libraries

@bradroid bradroid requested a review from Predjo October 12, 2022 10:52
@@ -30,7 +30,14 @@ buildscript {
}
}

plugins {
id "maven-publish"
id "digital.wup.android-maven-publish" version "3.6.2"
Copy link
Author

@bradroid bradroid Oct 12, 2022

Choose a reason for hiding this comment

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

If the AmityUIKit project used AGP 7.1 or newer we wouldn't need this dependency. Also, the scripts for attaching sources wouldn't be needed because we could simply use withSourcesJar() instead.

The project uses AGP 4.2.2 but I don’t think updating is a good idea.
If we update it might require us to migrate parts of the existing scripts which can be a lot of work.
Also, as Amity team updates their gradle scripts, they might not be compatible with the new AGP on our side so it can cause more work in the future when we’re pulling the new changes. Therefore I’ll leave it as is.

Copy link

@marinpiskacnoom marinpiskacnoom left a comment

Choose a reason for hiding this comment

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

LGTM

@bradroid bradroid merged commit 3bb5a7e into project-setup Oct 24, 2022
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