[Enhancement] Format existing code
See original GitHub issueSearch before asking
- I had searched in the issues and found no similar issues.
Description
Currently , there is so many code that isn’t formatted. It will make the review hard.
subtask WIP:
- checkstyle rule – redundant header https://github.com/apache/incubator-doris/pull/9072
- Add end tab charset rules for checkstyle – https://github.com/apache/incubator-doris/issues/9201
- Add
empty
rules for checkstyle https://github.com/apache/incubator-doris/issues/9203 - Add Simplify ArrayTypeStyle … rules for checkstyle https://github.com/apache/incubator-doris/issues/9202
- …
Solution
The solution need to discuss.
- we can format all code at once.
- we can format in batches, have priority to format the stable code.
------------------------- update ------------------------------
After I add checkstyle #9089 and fix them once #9113 .
I think we should fix code which can’t be fixed by IDEA-format
with checkstyle
at first.
So I add some subtask to do this thing.
Are you willing to submit PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project’s Code of Conduct
Issue Analytics
- State:
- Created a year ago
- Comments:5 (5 by maintainers)
Top Results From Across the Web
Source Code Formatter & Beautifier - TextMagic Free Tools
Use our free source code formatter to instantly transform, beautify, and format HTML, XML, CSS, JavaScript, PHP, or JSON code.
Read more >Automatically format and cleanup code every time you save
How to enable automatic formatting and cleanup · Go to Window > Preferences > Java > Editor > Save Actions. · Select Perform...
Read more >Clean Code: Formatting (Source code structure) | Nerd For Tech
Conclusion. Code formatting can increase the readability and interpretation of a code. At the start of the project, aim to identify and be ......
Read more >How do you auto format code in Visual Studio? - Stack Overflow
To format a selection: Ctrl + K , Ctrl + F. To format a document: Ctrl + K , Ctrl + D. See...
Read more >Formatting Code - Eclipse
To format your whole script: Open the required file. Go to Source | Format Document or press Ctrl+Shift+F. Your code will be automatically...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
The first thing is we should add a standard
fe/checkstyle.xml
. Which is need more discuss about what do we need rules.I want to add style checker to fe step by step. According to discussion in dev list, I’ll do this in six steps.
For detail, you could see: list thread: https://lists.apache.org/thread/yxjplc7v6sb0kzc9jxr1kl1xy2v8ctr3