Small issues reported by Codacy

[cassiosantos]

• Avoid unnecessary return statements
• Using equalsIgnoreCase() is cleaner than using toUpperCase/toLowerCase().equals()
• Avoid unused imports

[saijel]

Hi @cassiosantos, if you commented these imports out does than mean they are not used?

[KamalGalrani]

Those modules are used by a part of extractNEs function which is commented out. It's probably some debugging stuff. We can remove these lines as well as PrintWriter code from extractNEs function

[cassiosantos]

Lines removed

[jhauswald]

Generally I like this although we are planning on restructuring the repo and move services out. @cassiosantos do you have experience with continuous integration and specifically if generally people loop it in as part of that pipeline? curious to know your thoughts!

[cassiosantos]

@jhauswald restructuring the repository is a good idea.

Continuous integration comes with a set of good practices. I think it isn't necessary to use CI to take the advantage of automation.

I think that automated tests and builds could help people to contribute to the project.