Re: Add more rules to checkstyle.xml in fe

2022-06-14 Thread morrysnow
Hi , all devs After this PR: https://github.com/apache/incubator-doris/pull/10134 , all checkstyle error has been fixed. So, i want to turn Github workflow 'FE Code Style Checker' to 'Required' after this PR be merged. After #10134 be merge

Re:Re: Add more rules to checkstyle.xml in fe

2022-06-14 Thread morrysnow
Hi , all devs After this PR: https://github.com/apache/incubator-doris/pull/10134, all checkstyle error has been fixed. So, i want to turn Github workflow 'FE Code Style Checker' to 'Required' after this PR be merged. After #10134 be merged, when compiling fe with `maven`, `CheckStyle` checks

Re:Re: Add more rules to checkstyle.xml in fe

2022-06-14 Thread 陈明雨
I'm not sure if we currently have some overly restrictive rules that the developer hard to follow. However, I think we can try to turn on the required check, observe the recently submitted PR, and if any unreasonable checks are found, modify them in time to prevent such checks from affecting th

[GitHub] [incubator-doris-website] hf200012 merged pull request #10: update README.md to introduce blog and versions

2022-06-14 Thread GitBox
hf200012 merged PR #10: URL: https://github.com/apache/incubator-doris-website/pull/10 -- 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: dev-unsubscr...@d