Aqua.jl icon indicating copy to clipboard operation
Aqua.jl copied to clipboard

Code style check

Open YingboMa opened this issue 5 years ago • 1 comments

I wonder if it makes sense for Aqua.jl to check new trilling white spaces and tabs in a commit.

YingboMa avatar Jun 17 '20 03:06 YingboMa

I think it's a bit too non-trivial problem. You'd need CSTParser.jl if you try to handle string literals containing trailing spaces etc. I think a separate dedicated package might be better.

tkf avatar Jun 17 '20 04:06 tkf

I think code style checks are a bit out of bounds. For this, I'd recommend using a tool like JuliaFormatter (and e.g. the GitHub Action https://github.com/tkf/julia-code-style-suggesters by @tkf)

fingolfin avatar Jun 01 '23 12:06 fingolfin