devabhishekpal commented on code in PR #9684:
URL: https://github.com/apache/ozone/pull/9684#discussion_r2797210815
##########
dev-support/ci/selective_ci_checks.sh:
##########
@@ -441,6 +442,23 @@ function check_needs_pmd() {
start_end::group_end
}
+function check_needs_ui_lint() {
+ start_end::group_start "Check if UI lint is needed"
+ local pattern_array=(
+ "^hadoop-ozone/dev-support/checks/lint.sh"
+ "^ozone-ui"
+ "\\.tsx?$"
+ "\\.jsx?$"
+ )
+ filter_changed_files
Review Comment:
Yup this works now.
Changes have been addressed in the latest commit
```
❯ bats dev-support/ci/selective_ci_checks.bats
selective_ci_checks.bats
✓ UI code
✓ checkstyle and bats
✓ compose only
✓ dashboard only
✓ compose and robot
✓ runner image update
✓ check script
✓ java test + pmd change
✓ integration and unit: java change
✓ integration and unit: script change
✓ script change including javadoc.sh
✓ script change including junit.sh
✓ unit only
✓ unit helper
✓ integration only
✓ native only
✓ native test in other module
✓ kubernetes only
✓ docs only
✓ main/java change
✓ ..../java change
✓ java and compose change
✓ java and docs change
✓ pom change
✓ CI lib change
✓ CI workflow change
✓ draft CI workflow change
✓ CI workflow change (check.yml)
✓ CI workflow change (ci.yaml)
✓ root README
✓ ignored code
✓ PR-title workflow
✓ compose README
✓ other README
✓ LICENSE
✓ IntelliJ config
✓ dependency helper
✓ properties file in resources
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
