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

Migrate to gradle build system. #117

Merged
merged 3 commits into from
Feb 3, 2020
Merged

Conversation

ypbnv
Copy link
Contributor

@ypbnv ypbnv commented Jan 30, 2020

JIRA:
https://jira.appcelerator.org/browse/MOD-2563

Note:
I tried to get update to using AndroidX in the source files, but unfortunately it is not only a namespace change. I think it will bloat unnecessary this task and it will be for the best if we separate an update in another one.

@build
Copy link

build commented Jan 30, 2020

Messages
📖

💾 Here are the artifacts produced:

📖

✅ All tests are passing
Nice one! All 146 tests are passing.

Generated by 🚫 dangerJS against 79f6f20

android/build.gradle Outdated Show resolved Hide resolved
@sgtcoolguy
Copy link
Contributor

I think #112 should be considered before this one, seeing as it may impact things.

@jquick-axway
Copy link
Contributor

@sgtcoolguy, agreed. How about we go ahead and merge PR #112 and then test Android/iOS based on this PRs final changes?

Copy link
Contributor

@garymathews garymathews left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@garymathews garymathews merged commit 2b64671 into tidev:master Feb 3, 2020
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.

5 participants