Lint android's CMakeLists.txt
Depends on D4564
Differential D4568
[Chore] Lint native/android/app/CMakeLists.txt • jon on Jul 18 2022, 4:23 PM. Authored by Tags None Referenced Files
Details
Diff Detail
Event TimelineComment Actions Defining the dirs separately makes a lot of sense – I assume you did that manually @jonringer-comm, but will be very impressed with the Linter if it does that automatically! Separately, noticed that some of the lines here are longer than 80 chars, but for filenames I think it makes sense... it definitely hurts readability to split those across lines. Ideally we can have something that works like Prettier's printWidth setting, which generally limits line length but excludes import lines. Comment Actions
Strongly agree we shouldn't try splitting filenames in weird ways to stay within 80 char limit Comment Actions
I did it manually. The cmake-format command will mostly do whitespace and paren placement. |