Skip to content
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

Add .travis.yml to run tests and submit coverage to codecov.io #6

Merged
merged 1 commit into from May 10, 2018

Conversation

jirkadanek
Copy link

@jirkadanek jirkadanek commented May 10, 2018

The codecov.io report is (for now, since Travis is not enabled for this repository) at https://codecov.io/gh/jdanekrh/pooled-jms/tree/fc682b5da9c17351027f27f9a7830cccad359806

@tabish121
Copy link
Contributor

This doesn't seem like something that should be added here. The project includes tests against Apache ActiveMQ Artemis releases as well as ActiveMQ 5 releases and does it using standard unit testing methods that don't require brittle scripts and docker containers. Also the tests appear to be using snapshot which doesn't make a lot of sense either.

@jirkadanek
Copy link
Author

Whoops, yeah, that was silly of me. I've removed the docker part from the PR.

@jirkadanek jirkadanek changed the title Add .travis.yml to run tests against ActiveMQ Artemis broker in docker Add .travis.yml to run tests and submit coverage to codecov.io May 10, 2018
@tabish121 tabish121 merged commit 93e1945 into messaginghub:master May 10, 2018
@jirkadanek jirkadanek deleted the jd_travis branch May 10, 2018 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants