New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #1352 Add checkdoc to build #1957
Conversation
- fix doc strings as to checkdoc - add checkdoc to cider-checks - still missing nice way to make checkdoc ignore verb-tenses fixes clojure-emacs#1352
I think you should also add this to travis, right now it's not being ran automatically. |
@@ -61,7 +61,7 @@ | |||
|
|||
(defcustom cider-test-defining-forms '("deftest" "defspec") | |||
"Forms that define individual tests. | |||
CIDER considers the top-level form around point to define a test if the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you shouldn't be doing changes like this. Maybe quoting those words would be better. After all the terminology for this is top-level
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's already added to travis as I removed the comments around (dolist (file files)
(checkdoc-file file)) in |
Hmm. I checked the build and didn't see this anywhere in the output. Maybe
I missed something.
On Fri, Mar 10, 2017 at 14:40 Erik Assum ***@***.***> wrote:
I think it's already added to travis as I commented out
(dolist (file files)
(checkdoc-file file))
in test/cider-checks.el which to my understanding is what's run when make
test is run?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1957 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGVyindfDgKeDd6Lskt2ImvKk4SU453ks5rkUTXgaJpZM4MZTME>
.
--
Best Regards,
Bozhidar Batsov
http://www.batsov.com
|
I'm of the understanding that checkdocs is silent when it's happy. |
Perhaps. Just add some messages like |
Forget what I said. I just noticed it in the build. :-) |
This also fixes some issues checkdoc was currently reporting.
fixes #1352
make test
)M-x checkdoc
warnings