Build tools update (#911)

* Improve #877

* Update pull_request_template.md

* Update gradle version

* Update AGP version

* Update build files, use MavenPublisher plugin

* Add FAQs

* Fix tests

Author: natario1

.github/ISSUE_TEMPLATE/bug_report.md

@@ -13,11 +13,12 @@ Please add a clear description of what the bug is, **and** fill the list below.
 - Camera engine used: *camera1/camera2/both*
 - Reproducible in official demo app: *yes/no*
 - Device / Android version: *Pixel, API 28*
+- I have read the [FAQ page](https://natario1.github.io/CameraView/about/faq): *yes/no*
 
 ### To Reproduce
 Steps to reproduce the behavior, possibly in the demo app:
 1. Go to '...'
-2. Click on '....'
+2. Click on '...'
 3. See error
 
 ### Expected behavior

.github/ISSUE_TEMPLATE/question.md

@@ -8,7 +8,7 @@ assignees: ''
 ---
 
 ### How do I?
-Describe your problem here. Please, read the docs first.
+Describe your problem here. Please, read the [docs](https://natario1.github.io/CameraView) and [FAQ page](https://natario1.github.io/CameraView/about/faq) first.
 Questions not strictly related to CameraView should be asked elsewhere.
 
 ### Version used

.github/pull_request_template.md

@@ -1,10 +1,14 @@
 ### Before you go
-Unless this is a simple fix (typos, bugs with obvious solution), please open an issue first.
-If the edited files were covered by tests, updated tests are required for merging. Please look into the tests folders and make sure you cover new code.
+Unless this is a simple fix (typos, bugs with obvious solution), please open an issue first so that
+we can discuss the best approach to address the problem. Without a reference issue and discussion, 
+unfortunately, this PR will likely be ignored.
+
+If the edited files were covered by tests, updated tests are required for merging. 
+Please look into the tests folders and make sure you cover new code.
 
 - Fixes ... (*issue number*)
 - Tests: ... (*yes/no*)
 - Docs updated: ... (*yes/no*)
 
 ### Solution
-If applicable, briefly describe how the issue was addressed.
+If applicable, describe briefly how the issue was addressed.

.github/workflows/build.yml

@@ -18,7 +18,7 @@ jobs:
         with:
           java-version: 1.8
       - name: Perform base checks
-        run: ./gradlew demo:assembleDebug cameraview:javadoc
+        run: ./gradlew demo:assembleDebug cameraview:publishToDirectory
   ANDROID_UNIT_TESTS:
     name: Unit Tests
     runs-on: ubuntu-latest
@@ -28,12 +28,12 @@ jobs:
         with:
           java-version: 1.8
       - name: Execute unit tests
-        run: ./gradlew cameraview:testDebugUnitTest
+        run: ./gradlew cameraview:runUnitTests
       - name: Upload unit tests artifact
         uses: actions/upload-artifact@v1
         with:
           name: unit_tests
-          path: ./cameraview/build/jacoco/
+          path: ./cameraview/build/coverage_input/unit_tests
   ANDROID_EMULATOR_TESTS:
     name: Emulator Tests
     runs-on: macOS-latest
@@ -78,7 +78,7 @@ jobs:
         uses: actions/upload-artifact@v1
         with:
           name: emulator_tests_${{ matrix.EMULATOR_API }}
-          path: ./cameraview/build/outputs/code_coverage/debugAndroidTest/connected
+          path: ./cameraview/build/coverage_input/android_tests
   CODE_COVERAGE:
     name: Code Coverage Report
     runs-on: ubuntu-latest
@@ -92,23 +92,23 @@ jobs:
         uses: actions/download-artifact@v1
         with:
           name: unit_tests
-          path: ./cameraview/build/jacoco/
+          path: ./cameraview/build/coverage_input/unit_tests
       - name: Download emulator tests artifact
         uses: actions/download-artifact@v1
         with:
           # 27 is the EMULATOR_API with less SdkExclude annotations, and should have
           # the best possible coverage.
           name: emulator_tests_27
-          path: ./cameraview/build/outputs/code_coverage/debugAndroidTest/connected
+          path: ./cameraview/build/coverage_input/android_tests
       - name: Create merged coverage report
-        run: ./gradlew cameraview:mergeCoverageReports
+        run: ./gradlew cameraview:computeCoverage
       - name: Upload merged coverage report (GitHub)
         uses: actions/upload-artifact@v1
         with:
           name: report
-          path: ./cameraview/build/reports/mergedCoverageReport
+          path: ./cameraview/build/coverage_output/xml
       - name: Upload merged coverage report (Codecov)
         uses: codecov/codecov-action@v1
         with:
-          file: ./cameraview/build/reports/mergedCoverageReport/*
+          file: ./cameraview/build/coverage_output/xml/*
           fail_ci_if_error: true

.github/workflows/deploy.yml

@@ -11,10 +11,11 @@ jobs:
       TRAVIS: true
       BINTRAY_USER: ${{ secrets.BINTRAY_USER }}
       BINTRAY_KEY: ${{ secrets.BINTRAY_KEY }}
+      BINTRAY_REPO: ${{ secrets.BINTRAY_REPO }}
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-java@v1
         with:
           java-version: 1.8
       - name: Perform bintray upload
-        run: ./gradlew bintrayUpload
+        run: ./gradlew cameraview:publishToBintray

.github/workflows/emulator_script.sh

@@ -11,4 +11,4 @@ ADB_TAGS="$ADB_TAGS MediaEncoderEngine:I MediaEncoder:I AudioMediaEncoder:I Vide
 ADB_TAGS="$ADB_TAGS CameraIntegrationTest:I MessageQueue:W MPEG4Writer:I"
 adb logcat -c
 adb logcat $ADB_TAGS *:E -v color &
-./gradlew cameraview:connectedCheck
+./gradlew cameraview:runAndroidTests

build.gradle

@@ -0,0 +1,34 @@
+
+buildscript {
+
+    extra["minSdkVersion"] = 15
+    extra["compileSdkVersion"] = 29
+    extra["targetSdkVersion"] = 29
+    extra["kotlinVersion"] = "1.3.72"
+
+    repositories {
+        google()
+        mavenCentral()
+        jcenter()
+    }
+
+    dependencies {
+        classpath("com.android.tools.build:gradle:4.0.0")
+        classpath("com.otaliastudios.tools:publisher:0.3.3")
+        val kotlinVersion = property("kotlinVersion") as String
+        classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlinVersion")
+
+    }
+}
+
+allprojects {
+    repositories {
+        google()
+        mavenCentral()
+        jcenter()
+    }
+}
+
+tasks.register("clean", Delete::class) {
+    delete(buildDir)
+}