eranif / codelite

A multi purpose IDE specialized in C/C++/Rust/Python/PHP and Node.js. Written in C++
https://codelite.org
GNU General Public License v2.0
2.08k stars 444 forks source link

Move some files in sub directories #3412

Closed Jarod42 closed 7 hours ago

Eric-01 commented 3 days ago

Actually, I think it would be a good idea to run ClangFormat (and perhaps other formatters for other languages as well) on the entire codebase in one single commit, so that future commits don't contain 5 intended changes and 50 completely unrelated formatting improvements here and there made on the user's behalf by the formatter.

Other than that, I really appreciate the work you put into this project, as someone who hasn't contributed any new features or even bug fixes myself and just uses CodeLite for my own needs. Eranif can't keep up with everything (as seen in the delay when reviewing and merging pull requests), and you're doing a really good job 👍

Jarod42 commented 2 days ago

@Eric-01 : Running clang-format on whole repo should be done by Eranif directly (I think I even see a script in the repo) when no PRs is opened (or by CI bot, I have that in my TODO list for personal projects).

I try to minimize unrelated changes. Here, it is EOL consistency (mismatch \r\n \n between copyright header and regular source) that was changed by editor :-/