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.

Sign up for GitHub

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

Jump to bottom

Coveralls Support #42

Merged
merged 4 commits into from
Apr 25, 2019
Merged

Coveralls Support #42

merged 4 commits into from
Apr 25, 2019

Conversation

jklenzing
Copy link
Member

Closes #19.

Adds support for coveralls.

@jklenzing jklenzing added this to In progress in Unit Testing via automation Apr 24, 2019
@jklenzing jklenzing added the testing unit tests for code label Apr 24, 2019
@jklenzing jklenzing added this to the Beta Release milestone Apr 24, 2019
@jklenzing jklenzing self-assigned this Apr 24, 2019
@jklenzing
Copy link
Member Author

Badges currently linked to develop. Will update to master upon versioning.

@jklenzing jklenzing moved this from In progress to Needs review in Unit Testing Apr 25, 2019
JonathonMSmith
JonathonMSmith reviewed Apr 25, 2019
View reviewed changes
Copy link
Collaborator

@JonathonMSmith JonathonMSmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, made a change to resolve the conflict

@JonathonMSmith JonathonMSmith merged commit 3614bb8 into develop Apr 25, 2019
Unit Testing automation moved this from Needs review to Done Apr 25, 2019
@jklenzing jklenzing mentioned this pull request Apr 25, 2019
Closed
@jklenzing jklenzing deleted the coveralls branch May 11, 2019 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JonathonMSmith JonathonMSmith JonathonMSmith left review comments

Assignees

@jklenzing jklenzing

Labels
testing unit tests for code
Projects
No open projects
Unit Testing
  
Done
Milestone
0.1.0 Release
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jklenzing @JonathonMSmith