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

Is src/main/java/io/archivesunleashed/demo required? #17

Closed
ruebot opened this Issue Jul 8, 2017 · 5 comments

Comments

3 participants
@ianmilligan1

This comment has been minimized.

Member

ianmilligan1 commented Jul 8, 2017

I've never used it - @lintool?

@lintool

This comment has been minimized.

Member

lintool commented Jul 10, 2017

I think we can axe this for now, but we should add end-to-end integration tests along these lines at some point...

@ruebot

This comment has been minimized.

Member

ruebot commented Jul 10, 2017

Cool, I'll yank that out as part of #3.

My goal is to get the test coverage up to high 90%, and maybe also setup Jenkins and publish SNAPSHOTs if need be.

@lintool

This comment has been minimized.

Member

lintool commented Jul 10, 2017

sg

ruebot added a commit that referenced this issue Jul 17, 2017

Add checkstyle plugin, and remove demo; resolves #3 #17.
* Add checkstyle plugin
* Add suppression files for checkstyle
* Address checkstyle warnings
* Add javadocs where necessary
* Remove demo

lintool added a commit that referenced this issue Jul 31, 2017

Add checkstyle plugin, and remove demo; resolves #3 #17. (#19)
+ Add checkstyle plugin
+ Add suppression files for checkstyle
+ Address checkstyle warnings
+ Add javadocs where necessary
+ Remove demo
@ruebot

This comment has been minimized.

Member

ruebot commented Jul 31, 2017

Resolved with bb2e665

@ruebot ruebot closed this Jul 31, 2017

@ruebot ruebot added this to Done in 1.0.0 Release of AUT Aug 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment