RE: build tool suggestion

2015-05-06 Thread Finan, Sean
Your IDE should have settings that allow custom warnings. Also check out findbugs -- http://en.wikipedia.org/wiki/FindBugs There might be a configurable maven plugin. It is a process ... -Original Message- From: Masanz, James J. [mailto:masanz.ja...@mayo.edu] Sent: Tuesday, May 05,

cTAKES - Smoking Status

2015-05-06 Thread Bohne, Jacqueline R
We are trying to implement the Smoking Status pipeline at our site. We have it up and running, but we're finding that TOBACCO: NONE is being incorrectly classified as a CURRENT_SMOKER. I looked at the documentation and it appears that the SentenceAdjuster should be handling this situation? I

RE: build tool suggestion

2015-05-06 Thread Finan, Sean
I understood that. I check warnings before checkin. You can do a search for something like https://wiki.jenkins-ci.org/display/JENKINS/Warnings+Plugin -Original Message- From: Masanz, James J. [mailto:masanz.ja...@mayo.edu] Sent: Wednesday, May 06, 2015 10:58 AM To:

Re: build tool suggestion

2015-05-06 Thread Kim Ebert
Would this only be for the Jenkins build? I would prefer not to have this as part of the regular Maven build. On Tue, May 5, 2015 at 6:00 PM, Masanz, James J. masanz.ja...@mayo.edu wrote: Do you know offhand, would it be easy to have something run at build time that flags uses of FileReader?

RE: build tool suggestion

2015-05-06 Thread Masanz, James J.
Sorry, I wasn't clear, when I said at build time, I meant the Jenkins automated build. -Original Message- From: Finan, Sean [mailto:sean.fi...@childrens.harvard.edu] Sent: Wednesday, May 06, 2015 9:52 AM To: dev@ctakes.apache.org Subject: RE: build tool suggestion Your IDE should

RE: build tool suggestion

2015-05-06 Thread Masanz, James J.
I was suggesting it only for the Jenkins build, so that people who might not want it as part of their own builds won't see it. But to catch people who might not have their own warning individually defined or who miss it in the warnings of their own build. -Original Message- From: Kim