-
-
Notifications
You must be signed in to change notification settings - Fork 293
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
Adding CI / CD for Processing android mode #707
Comments
Is there any idea to elaborate or process to implement for Continous Integration? |
With CI-enabled merging PR will be very easy for reviewers. Githu CI can be utilized here, it's free, fast and easy to customize. Here's an example |
Hi @Chandra-Sekhar-Bala , currently maintainers test all the changes on local machines before merging single line of code into tha main, because main brach is big and contains core-processing-android, vr-lib, ar-lib and scripts as well. What you can do if you want to make the building process automated so whenever someone make a PR CI process build that PR branch's code and shpw if its mergeable or not - you can make a CI / CD pipeline and integrate with processing-android. What I can inderstand from your question is you want to make CI process for android-mode so that maintainbers need not to build it locally everytime, right ? Thanks, |
What's your say @codeanticode on this, I was planning to add CI building process, linting, code build checker, and welcome comments automation for android-mode as well like p5.js Thanks ! |
@ranaaditya yes I think that having those elements of CI would be great. I have not done it before, but it would be a great addition to make the contribution process better structured. |
@ranaaditya You're correct, that's what I am trying to achieve here. Also, contributors will be able to see why their PR can/can't be merged, when && what goes wrong! |
@Chandra-Sekhar-Bala you are good to go here ! let me know if you need any help in this issue, Thanks, |
What is the minSDK we should check ? @ranaaditya |
@Chandra-Sekhar-Bala We currently target android 30, but we support the following min SDK versions depending on the type of app:
Does this answer your quesiton? |
I tried to look into the codebase, didn't understand a bit. Sorry, I don't have any experience in this type of codebase. I am familiar with a repo including app folder, but I am willing to learn. Please guide me through it, I really want to contribute. @ranaaditya @rupeshkumar22 @codeanticode |
@Chandra-Sekhar-Bala You can first try to get familiar with Android Mode on PDE to get to know what functionalities it provides to build an android app based on processing. You can get started with https://android.processing.org/install.html Then you can try to build the source code of Android Mode along with Processing4 - https://github.com/processing/processing-android/wiki/Building-Processing-for-Android You can do some changes in the code base, then build, and then test. There are a couple of issues available on this repo to start with. Let me know if you get stuck somewhere. All the best! |
Thanks @rupeshkumar22. Thanks for guiding, I will follow that. |
Hi @Chandra-Sekhar-Bala , processing-android is neither an android app nor a single android library, its collection of android-core, vr, ar and utility libraries. To start automating the building and checking process you can follow something like:
In this way you can start the automating process for this project. NOTE: you need not to learn the processing APIs or how to make a sketch as this task is not related to developing processing-core or utility libs used in this project as I believe automation will not touch any existing core files and will also not going to touch the build.gradle files. In case of any doubt just tagg me here or we can also schedule a meet based on our availablity. Thanks, |
Adding few resources if someone picks this issue in future. Building a CI-CD Pipeline for an Android Thanks, |
If
pull request
is created to themaster
branch, check whether this PR has any problem or not. it is good to make a CI process.The text was updated successfully, but these errors were encountered: