Fix #1736: Add Bazel Linter

.github/workflows/main.yml

@@ -11,7 +11,7 @@ on:
       - develop
 
 # This workflow has the following jobs:
-# linters: Java, Kotlin, Protobuf
+# linters: Java, Kotlin, Protobuf, Bazel
 # robolectric_tests: Robolectric tests for all modules except the app module
 # app_tests: Non-flaky Robolectric tests for the app module
 jobs:
@@ -52,6 +52,11 @@ jobs:
         curl -sSL https://github.com/bufbuild/buf/releases/download/$BUFVERSION/buf-$(uname -s)-$(uname -m) > buf
         chmod a+x buf
 
+    - name: Download Buildifier
+      run: |
+        cd $HOME/oppia-android-tools
+        bash /home/runner/work/oppia-android/oppia-android/scripts/buildifier_download.sh
+
     - name: Java lint check
       run: |
         bash /home/runner/work/oppia-android/oppia-android/scripts/checkstyle_lint_check.sh $HOME
@@ -63,6 +68,10 @@ jobs:
     - name: Protobuf lint check
       run: $HOME/oppia-android-tools/buf check lint --input=model/src/main/proto --input-config buf.yaml && echo "Protobuf lint check completed successfully"
 
+    - name: Bazel lint check
+      run: |
+        bash /home/runner/work/oppia-android/oppia-android/scripts/buildifier_lint_check.sh $HOME
+
   robolectric_tests:
     name: Robolectric Tests (Non-App Modules)
     runs-on: ${{ matrix.os }}

BUILD.bazel

@@ -7,10 +7,10 @@ android_binary(
     manifest = "//app:src/main/AndroidManifest.xml",
     manifest_values = {
         "applicationId": "org.oppia.android",
-        "versionCode": "0",
-        "versionName": "0.1-alpha",
         "minSdkVersion": "19",
         "targetSdkVersion": "29",
+        "versionCode": "0",
+        "versionName": "0.1-alpha",
     },
     multidex = "native",  # TODO(#1875): Re-enable legacy for optimized release builds.
     deps = [

WORKSPACE

@@ -2,9 +2,8 @@
 This file lists and imports all external dependencies needed to build Oppia Android.
 """
 
-load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
-load("@bazel_tools//tools/build_defs/repo:jvm.bzl", "jvm_maven_import_external")
 load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository")
+load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
 load("//third_party:versions.bzl", "HTTP_DEPENDENCY_VERSIONS", "get_maven_dependencies")
 
 # Android SDK configuration. For more details, see:
@@ -41,12 +40,10 @@ kotlin_repositories()
 
 kt_register_toolchains()
 
-"""
-The proto_compiler and proto_java_toolchain bindings load the protos rules needed for the model
-module while helping us avoid the unnecessary compilation of protoc. Referecences:
-- https://github.com/google/startup-os/blob/5f30a62/WORKSPACE#L179-L187
-- https://github.com/bazelbuild/bazel/issues/7095
-"""
+# The proto_compiler and proto_java_toolchain bindings load the protos rules needed for the model
+# module while helping us avoid the unnecessary compilation of protoc. Referecences:
+# - https://github.com/google/startup-os/blob/5f30a62/WORKSPACE#L179-L187
+# - https://github.com/bazelbuild/bazel/issues/7095
 
 bind(
     name = "proto_compiler",