Merge master into phobos#482
Conversation
A check for public declaration without a documented unittest.
* Enable DScanner checking on Travis * Fix linter warnings * Set opequals_tohash_check to disabled * Set could_be_immutable to 'disabled' * Split expression back into multiple lines * Set style to disabled * Fix Makefile target
Fix AssertError on unknown label in unused_label merged-on-behalf-of: Sebastian Wilzbach <sebi.wilzbach@gmail.com>
…c_example Add has_public_example check
|
Thanks for your pull request, @wilzbach! |
|
I see that you enabled the useless initializer but
|
This is just a periodic sync from master to phobos
Will have a look into finally pushing it.
It's disabled at Phobos: https://github.com/dlang/phobos/blob/e4afacadb8a291f51444e948dd7b53592e799dbe/.dscanner.ini#L156 |
|
I don't understand, what do you use this branch for ? |
Sorry that I didn't explain - merging & testing features like #460 without needing to wait for review & approval. Inversely, Phobos has proven to be a good battlefield / playground to test new checks. So the intention is also to speed up reviewing by (1) improving a check, (2) showing that it works in real life. A version of the |
|
Ok now i see that you filter std for the useless init check. Thanks for the explanations. |
2 participants
Keeping the phobos branch up-to-date.
phobosan experimental branch of Dscanner in which I can safely apply new features to be rolled out for Phobos.