aboutsummaryrefslogtreecommitdiffhomepage
path: root/build_tools/style.fish
Commit message (Collapse)AuthorAge
* Avoid using chmod --reference in style.fishGravatar ridiculousfish2016-06-11
| | | | | | chmod --reference is not available on OS X Instead, we copy the source file into the temporary path, so that mode bits are preserved
* lint, style: use git plumbing commandsGravatar David Adam2016-05-18
| | | | | Rather than using porcelain commands, try using plumbing for a more stable interface with less string munging.
* only deal with files that existGravatar Kurtis Rader2016-05-08
| | | | | I noticed while working on an unrelated change that deleting a file caused `make lint` to behave in an unexpected manner.
* limit `make style-all` to fish scripts in shareGravatar Kurtis Rader2016-05-03
| | | | | | | | | I noticed that if I've previous done `make test` that a subsequent `make style-all` attempts to restyle all the fish scripts in the *test* directory. Those files are transient and not part of the git repository. Limit restyling all fish scripts just to those in the *share* directory tree. There are a couple elsewhere in the repo (e.g., *build_tools*) but they can be handled on an individual basis.
* restyle builtin modules to match project styleGravatar Kurtis Rader2016-04-26
| | | | | | | | | | | Now that the IWYU cleanup has been merged compile all, not just a couple, of the builtin modules independent of builtin.cpp. That is, no longer `#include builtin_NAME.cpp` in builtin.cpp. This is more consistent, more in line with what developers expect, and is likely to reduce mistakes. Reduces lint errors from 384 to 336 (-13%). Line count from 6307 to 4988 (-21%). Another step in resolving issue #2902.
* improve the style.fish scriptGravatar Kurtis Rader2016-04-12
| | | | | | | | | | If there are uncommitted changes use `git-clang-format` to limit the style fixups to the lines being modified. Refuse to do a `make style-all` if there are uncommitted changes. Include a fix for the parsing of `git status` output that was recently incorporated into the lint.fish script.
* support making fish code match the style guideGravatar Kurtis Rader2016-04-04
This changes implements two new make targets: `style` and `style-all`. These make it easy to ensure that a change conforms to the project style guides for C++ and fish code. Fixes #571