Skip to content

Commit 34707d8

Browse files
panbingkunjingz-db
authored andcommitted
[SPARK-48763][FOLLOWUP] Make dev/lint-scala error message more accurate
### What changes were proposed in this pull request? The pr is followuping apache#47157, to make `dev/lint-scala` error message more accurate. ### Why are the changes needed? After move from: `connector/connect/server` `connector/connect/common` to: `connect/server``connect/common` Our error message in `dev/lint-scala` should be updated synchronously. eg: <img width="709" alt="image" src="https://github.com/apache/spark/assets/15246973/d749e371-7621-4063-b512-279d0690d573"> <img width="772" alt="image" src="https://github.com/apache/spark/assets/15246973/44b80571-bdb6-40cb-9571-8b34d009b5f8"> ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GA. Manually test. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#47291 from panbingkun/SPARK-48763_FOLLOWUP. Authored-by: panbingkun <panbingkun@baidu.com> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
1 parent c88a8cd commit 34707d8

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

dev/lint-scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ ERRORS=$(./build/mvn \
3636
)
3737

3838
if test ! -z "$ERRORS"; then
39-
echo -e "The scalafmt check failed on connector/connect at following occurrences:\n\n$ERRORS\n"
39+
echo -e "The scalafmt check failed on connect or connector/connect at following occurrences:\n\n$ERRORS\n"
4040
echo "Before submitting your change, please make sure to format your code using the following command:"
4141
echo "./build/mvn scalafmt:format -Dscalafmt.skip=false -Dscalafmt.validateOnly=false -Dscalafmt.changedOnly=false -pl connect/common -pl connect/server -pl connector/connect/client/jvm"
4242
exit 1

0 commit comments

Comments
 (0)