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

Fix #3304: Add check to ensure KDoc presence for non-private members #3499

Merged
merged 203 commits into from
Jul 30, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
203 commits
Select commit Hold shift + click to select a range
225a4fc
Fix #3290: Add support for generic regex pattern matching
Sparsh1212 Jun 10, 2021
638895f
Apply review suggestions for proto location
Sparsh1212 Jun 11, 2021
e3fd706
Delete main directory
Sparsh1212 Jun 11, 2021
b25e776
Implment script logic and tests
Sparsh1212 Jun 14, 2021
d44e015
CI setup part 1
Sparsh1212 Jun 14, 2021
de1d38f
syntax fix in yaml file
Sparsh1212 Jun 14, 2021
173fe53
import nits in dummy file
Sparsh1212 Jun 15, 2021
618e380
Use regex patterns from script in script test
Sparsh1212 Jun 16, 2021
e3830ac
Make PR suggestions into effect
Sparsh1212 Jun 16, 2021
bb93c0a
Make PR suggestions into effect
Sparsh1212 Jun 16, 2021
f7342c1
Make PR suggestions into effect
Sparsh1212 Jun 16, 2021
51e593b
Improve naming of script_assets variables
Sparsh1212 Jun 16, 2021
0d17657
Make PR suggestions into effect
Sparsh1212 Jun 16, 2021
16bb041
Make PR suggestions
Sparsh1212 Jun 17, 2021
325f28c
Make PR suggestions
Sparsh1212 Jun 17, 2021
a3f8ea6
Revamp testing approach
Sparsh1212 Jun 18, 2021
b1970eb
Merge branch 'develop' into regex-checks
Sparsh1212 Jun 18, 2021
51e64aa
Fix #3291: Add check for XML syntax correctness
Sparsh1212 Jun 18, 2021
f440a33
nit fixes
Sparsh1212 Jun 18, 2021
255773d
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 18, 2021
7b2a178
Introduce XML syntax check in static checks workflow
Sparsh1212 Jun 18, 2021
14c5f6f
Fix #3292: Add check for test files presence for prod files
Sparsh1212 Jun 18, 2021
163310e
Make nit suggestions
Sparsh1212 Jun 20, 2021
6131c1d
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 21, 2021
9cb5fc5
Make nit suggestions
Sparsh1212 Jun 21, 2021
4120775
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 21, 2021
8c98cf2
Make nit suggestions
Sparsh1212 Jun 21, 2021
7f1762c
Fix #3300: Add check for accessibility labels for activities
Sparsh1212 Jun 21, 2021
5229ac9
Add CI check for label presence for activities
Sparsh1212 Jun 21, 2021
7e1d24a
Add ScriptConstants file to exemptions test list
Sparsh1212 Jun 21, 2021
08cf6c3
merge test-file-check
Sparsh1212 Jun 21, 2021
b72e419
Test if: always()
Sparsh1212 Jun 21, 2021
d8472ab
Revert "Test if: always()"
Sparsh1212 Jun 21, 2021
fe42a64
Add if: always()
Sparsh1212 Jun 21, 2021
e12f247
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 21, 2021
9fdb165
Add if: always()
Sparsh1212 Jun 21, 2021
7bf2726
Merge branch 'test-file-check' into accesibilty-label-check
Sparsh1212 Jun 21, 2021
6e3424b
Add if: always()
Sparsh1212 Jun 21, 2021
381853c
Apply review suggestions on PR
Sparsh1212 Jun 22, 2021
c59494f
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 23, 2021
293ab92
Make review suggestions from regex pattern checks
Sparsh1212 Jun 23, 2021
f393e1e
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 23, 2021
56d607e
Implement review suggestions based on #3340
Sparsh1212 Jun 24, 2021
d01a238
Refactor PR as per feedback recieved
Sparsh1212 Jun 24, 2021
b7cb290
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 24, 2021
5a4ac03
Implement feedback suggestions
Sparsh1212 Jun 24, 2021
f6b0715
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 24, 2021
0feacee
Implement review suggestions
Sparsh1212 Jun 24, 2021
28fbc73
Implement review suggestions
Sparsh1212 Jun 24, 2021
96c0bfd
Implement review suggestions
Sparsh1212 Jun 24, 2021
dcbf660
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 24, 2021
56cf6d0
Implement review suggestions
Sparsh1212 Jun 24, 2021
1e3c47d
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 24, 2021
60e64dc
nits
Sparsh1212 Jun 25, 2021
cd2f287
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 25, 2021
36033ef
Nit fixes
Sparsh1212 Jun 25, 2021
161f8ec
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 25, 2021
996694b
Nit fixes
Sparsh1212 Jun 25, 2021
8740a48
Nit fixes
Sparsh1212 Jun 25, 2021
6671857
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 25, 2021
11963c8
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 25, 2021
c8aa5e4
merge test-file-check
Sparsh1212 Jun 26, 2021
3b78452
Implement feedback suggestions as per #3340
Sparsh1212 Jun 26, 2021
fe17ed9
nit fix
Sparsh1212 Jun 27, 2021
6860083
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jun 27, 2021
4083244
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 27, 2021
7c6f3e8
Merge branch 'test-file-check' into accesibilty-label-check
Sparsh1212 Jun 27, 2021
d65efa4
nit fix
Sparsh1212 Jun 27, 2021
3afc018
nit fix
Sparsh1212 Jun 28, 2021
b13e3f3
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jun 28, 2021
6132e5d
Merge branch 'test-file-check' into accesibilty-label-check
Sparsh1212 Jun 28, 2021
4a5b47e
nit fix
Sparsh1212 Jun 28, 2021
60f2482
Review suggestions part 1
Sparsh1212 Jun 29, 2021
9375775
Implement review suggestions part 2
Sparsh1212 Jun 29, 2021
a94cc5e
nit fixes
Sparsh1212 Jun 30, 2021
a811521
resolve merge conflict
Sparsh1212 Jun 30, 2021
aeb20d3
update static_checks
Sparsh1212 Jun 30, 2021
d5a0166
add test to repository file
Sparsh1212 Jun 30, 2021
8ac402d
nit fix
Sparsh1212 Jun 30, 2021
32910ea
nit fix
Sparsh1212 Jun 30, 2021
8b749db
nit fix
Sparsh1212 Jun 30, 2021
6c08868
merge regex-checks
Sparsh1212 Jun 30, 2021
4add1ed
Do review suggestions
Sparsh1212 Jun 30, 2021
be2f808
Implement review suggestions
Sparsh1212 Jul 1, 2021
1002034
Implement review suggestions
Sparsh1212 Jul 1, 2021
11010b1
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 1, 2021
ad4b0ec
nit fix
Sparsh1212 Jul 1, 2021
8eaf8df
merge xml-syntax-check
Sparsh1212 Jul 1, 2021
3042f2e
Implement review suggestions
Sparsh1212 Jul 1, 2021
49745a4
Add Ben as a codeowner for the ScriptExemptions file
Sparsh1212 Jul 1, 2021
05dbe3e
bazel files nit fix
Sparsh1212 Jul 1, 2021
45d73c9
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 1, 2021
5a9c601
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 1, 2021
83d66e0
Introduce a library for the regex assets
Sparsh1212 Jul 1, 2021
8cae1f8
Make directory structure same as that of #3374
Sparsh1212 Jul 2, 2021
e201a3d
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 2, 2021
44d839c
Make directory structure similar to #3374
Sparsh1212 Jul 2, 2021
edd6b4e
Merge branch 'develop' into regex-checks
Sparsh1212 Jul 2, 2021
5e38f79
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 2, 2021
73de958
merge xml-syntax-checks
Sparsh1212 Jul 2, 2021
9e67cab
add new files to test file script exemptions and nit fixes
Sparsh1212 Jul 2, 2021
7a73b1f
merge develop
Sparsh1212 Jul 3, 2021
1637ccd
diable ktlint max-line-length
Sparsh1212 Jul 3, 2021
fba990e
disable ktlint-max-line
Sparsh1212 Jul 3, 2021
262a4ec
disable ktlint max-length
Sparsh1212 Jul 3, 2021
8e5887c
merge regex-checks
Sparsh1212 Jul 3, 2021
fcb0555
remove testonly attribute from tests
Sparsh1212 Jul 3, 2021
de21e33
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 3, 2021
130c0d9
add new files to script exemptions
Sparsh1212 Jul 3, 2021
b56c104
nit fix
Sparsh1212 Jul 4, 2021
cdb88fe
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 4, 2021
98f7d73
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 4, 2021
3459461
nit fix
Sparsh1212 Jul 5, 2021
69c5f96
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 5, 2021
26a8d52
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 5, 2021
2e785d5
merge regex-checks
Sparsh1212 Jul 5, 2021
5e13219
nit fix
Sparsh1212 Jul 5, 2021
21e5197
Apply review suggestions
Sparsh1212 Jul 6, 2021
e557570
ktlint fix
Sparsh1212 Jul 6, 2021
3cb5db3
nit fix
Sparsh1212 Jul 6, 2021
c37cca3
Apply nit fixes
Sparsh1212 Jul 6, 2021
e2186d7
Apply nit fixes
Sparsh1212 Jul 6, 2021
7d7f9fc
Remove script constants
Sparsh1212 Jul 7, 2021
b70c880
nit fix
Sparsh1212 Jul 7, 2021
e70413b
add tests for XML syntax error handler
Sparsh1212 Jul 7, 2021
8e8bcf5
merge xml-syntax-check
Sparsh1212 Jul 7, 2021
31c1df5
add todo
Sparsh1212 Jul 7, 2021
5849bc0
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 7, 2021
e09e837
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 7, 2021
637606c
add todo
Sparsh1212 Jul 7, 2021
5ed9d9a
Merge branch 'develop' into regex-checks
Sparsh1212 Jul 8, 2021
bab0b70
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 8, 2021
c30158c
nit fixes
Sparsh1212 Jul 8, 2021
d57ef42
Merge branch 'develop' into regex-checks
Sparsh1212 Jul 8, 2021
d873ecd
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 8, 2021
de8bf3a
add test
Sparsh1212 Jul 8, 2021
5a1cbd9
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 8, 2021
91e108a
add testOnly
Sparsh1212 Jul 8, 2021
17ddf56
Merge branch 'regex-checks' into xml-syntax-check
Sparsh1212 Jul 8, 2021
594d31a
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 8, 2021
1940980
nit fixes
Sparsh1212 Jul 8, 2021
d34d46b
nit fix
Sparsh1212 Jul 8, 2021
ba0b960
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 8, 2021
50ea8ae
nit fixes
Sparsh1212 Jul 8, 2021
c72e5a9
nit fixes
Sparsh1212 Jul 8, 2021
57d0b42
nit fixes
Sparsh1212 Jul 8, 2021
c32c921
nit fixes
Sparsh1212 Jul 8, 2021
b99a316
fix textproto file
Sparsh1212 Jul 8, 2021
af6a607
merge-regex-checks
Sparsh1212 Jul 8, 2021
6bdea69
Merge branch 'xml-syntax-check' into test-file-check
Sparsh1212 Jul 8, 2021
7f13ff8
merge xml-syntax-check
Sparsh1212 Jul 8, 2021
19fc1e9
add exemptions to textproto
Sparsh1212 Jul 8, 2021
99a1f13
sort failures lexicographically
Sparsh1212 Jul 8, 2021
0dd079c
Merge branch 'develop' into test-file-check
Sparsh1212 Jul 9, 2021
f286c73
add new files to exemption list
Sparsh1212 Jul 9, 2021
0db928c
merge test-file-check
Sparsh1212 Jul 11, 2021
0d2d9dd
merge develop
Sparsh1212 Jul 11, 2021
a331ffa
nit fixes
Sparsh1212 Jul 11, 2021
484891e
nit fixes
Sparsh1212 Jul 11, 2021
5a7495d
add kdocs
Sparsh1212 Jul 13, 2021
9988373
nit fix
Sparsh1212 Jul 13, 2021
0eb4dd6
nit fix
Sparsh1212 Jul 13, 2021
76b8e2c
Trigger CI failure exclusively
Sparsh1212 Jul 14, 2021
e34798e
Revert CI check failure
Sparsh1212 Jul 14, 2021
1fdf50e
Fix #3340: Create a script to ensure KDoc presence
Sparsh1212 Jul 14, 2021
2fa1af8
Add initial tests
Sparsh1212 Jul 17, 2021
023bac6
Add more tests and add documentation
Sparsh1212 Jul 17, 2021
75bb351
nit fix
Sparsh1212 Jul 17, 2021
dcba69f
merge develop
Sparsh1212 Jul 17, 2021
f09d0d4
nit fix
Sparsh1212 Jul 17, 2021
9c94456
nit fix
Sparsh1212 Jul 17, 2021
03576d1
merge accessibility label check
Sparsh1212 Jul 17, 2021
633cb4b
nit fix
Sparsh1212 Jul 17, 2021
34ad4e9
Add more files to kdoc exemptions
Sparsh1212 Jul 17, 2021
b042bba
nit fixes
Sparsh1212 Jul 20, 2021
3db1d29
nit fixes
Sparsh1212 Jul 20, 2021
8e9c383
Merge branch 'develop' into accesibilty-label-check
Sparsh1212 Jul 22, 2021
dc2b163
nit fixes
Sparsh1212 Jul 22, 2021
b012d04
nit fixes
Sparsh1212 Jul 22, 2021
eacacba
Add redundant exemptions check
Sparsh1212 Jul 24, 2021
57a4b8d
Merge branch 'develop' into accesibilty-label-check
Sparsh1212 Jul 24, 2021
43cf464
rectify static_checks
Sparsh1212 Jul 24, 2021
3d3b03f
Remove duplicate proto libraries
Sparsh1212 Jul 24, 2021
26df126
merge accessibility-check
Sparsh1212 Jul 24, 2021
4082259
nit fixes
Sparsh1212 Jul 24, 2021
8db6c21
nit fixes and add redundant exemptions check to script
Sparsh1212 Jul 25, 2021
1aa7ead
nit fixes
Sparsh1212 Jul 28, 2021
9fb4e1c
add activity to exemption
Sparsh1212 Jul 28, 2021
94a482b
Merge branch 'accesibilty-label-check' into kdoc-check
Sparsh1212 Jul 28, 2021
8506662
nit fixes
Sparsh1212 Jul 28, 2021
fffef74
nit fixes
Sparsh1212 Jul 28, 2021
2ed0719
Merge branch 'accesibilty-label-check' into kdoc-check
Sparsh1212 Jul 28, 2021
a7efb89
nit fixes
Sparsh1212 Jul 28, 2021
6f0f05c
nit fixes
Sparsh1212 Jul 28, 2021
6c4c64f
Merge branch 'accesibilty-label-check' into kdoc-check
Sparsh1212 Jul 28, 2021
015d717
nit fixes
Sparsh1212 Jul 28, 2021
fda44e4
Merge branch 'accesibilty-label-check' into kdoc-check
Sparsh1212 Jul 28, 2021
cb67d44
Merge branch 'develop' into accesibilty-label-check
Sparsh1212 Jul 28, 2021
bb68dcd
update branch
Sparsh1212 Jul 28, 2021
2343423
Merge branch 'accesibilty-label-check' into kdoc-check
Sparsh1212 Jul 28, 2021
a397534
update branch
Sparsh1212 Jul 30, 2021
a46a95c
Merge branch 'develop' into kdoc-check
Sparsh1212 Jul 30, 2021
c55a8cf
nit fixes
Sparsh1212 Jul 30, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/workflows/static_checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -123,3 +123,8 @@ jobs:
if: always()
run: |
bazel run //scripts:accessibility_label_check -- $(pwd) scripts/assets/accessibility_label_exemptions.pb app/src/main/AndroidManifest.xml

- name: KDoc Validation Check
if: always()
run: |
bazel run //scripts:kdoc_validity_check -- $(pwd) scripts/assets/kdoc_validity_exemptions.pb
16 changes: 16 additions & 0 deletions scripts/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ load("@io_bazel_rules_kotlin//kotlin:kotlin.bzl", "kt_jvm_binary", "kt_jvm_libra
load(
"//scripts:script_assets.bzl",
"generate_accessibility_label_assets_list_from_text_protos",
"generate_kdoc_validity_assets_list_from_text_protos",
"generate_maven_assets_list_from_text_protos",
"generate_regex_assets_list_from_text_protos",
"generate_test_file_assets_list_from_text_protos",
Expand Down Expand Up @@ -131,3 +132,18 @@ kt_jvm_binary(
"//scripts/src/java/org/oppia/android/scripts/label:accessibility_label_check_lib",
],
)

KDOC_VALIDITY_EXEMPTION_ASSETS = generate_kdoc_validity_assets_list_from_text_protos(
name = "kdoc_validity_exemption_assets",
kdoc_validity_exemptions_name = ["kdoc_validity_exemptions"],
)

kt_jvm_binary(
name = "kdoc_validity_check",
testonly = True,
data = KDOC_VALIDITY_EXEMPTION_ASSETS,
main_class = "org.oppia.android.scripts.docs.KdocValidityCheckKt",
runtime_deps = [
"//scripts/src/java/org/oppia/android/scripts/docs:kdoc_validity_check_lib",
],
)
626 changes: 626 additions & 0 deletions scripts/assets/kdoc_validity_exemptions.textproto

Large diffs are not rendered by default.

24 changes: 24 additions & 0 deletions scripts/script_assets.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -112,3 +112,27 @@ def generate_accessibility_label_assets_list_from_text_protos(
proto_dep_bazel_target_prefix = "//scripts/src/java/org/oppia/android/scripts/proto",
proto_package = "proto",
)

def generate_kdoc_validity_assets_list_from_text_protos(
name,
kdoc_validity_exemptions_name):
"""
Converts a single list of text proto assets to binary.

Args:
name: str. The name of this generation instance. This will be a prefix for derived targets.
kdoc_validity_exemptions_name: list of str. The list of kdoc validity exemptions file name.

Returns:
list of str. The list of new proto binary asset files that were generated.
"""
return generate_proto_binary_assets(
name = name,
names = kdoc_validity_exemptions_name,
proto_dep_name = "script_exemptions",
proto_type_name = "KdocValidityExemptions",
name_prefix = name,
asset_dir = "assets",
proto_dep_bazel_target_prefix = "//scripts/src/java/org/oppia/android/scripts/proto",
proto_package = "proto",
)
17 changes: 17 additions & 0 deletions scripts/src/java/org/oppia/android/scripts/docs/BUILD.bazel
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
"""
Libraries corresponding to documentation-related scripts.
"""

load("@io_bazel_rules_kotlin//kotlin:kotlin.bzl", "kt_jvm_library")

kt_jvm_library(
name = "kdoc_validity_check_lib",
testonly = True,
srcs = ["KdocValidityCheck.kt"],
visibility = ["//scripts:oppia_script_binary_visibility"],
deps = [
"//scripts/src/java/org/oppia/android/scripts/common:repository_file",
"//scripts/src/java/org/oppia/android/scripts/proto:script_exemptions_java_proto_lite",
"//third_party:org_jetbrains_kotlin_kotlin-compiler-embeddable",
],
)
Loading