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

Test coverage and package updates #4

Merged
merged 1 commit into from Apr 25, 2016
Merged

Conversation

wehriam
Copy link
Collaborator

@wehriam wehriam commented Apr 25, 2016

Description

  • Travis CI setup
  • Extended test coverage
  • Update packages to latest versions

Motivation and Context

The SuperScript components have limited test coverage, making updates dangerous and difficult.

How Has This Been Tested?

These changes updated the tests themselves.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Version bump to trigger Travis build.

Add badges.

Expand test coverage. Create and load using callbacks. Minor style updates.

Issue and pull request templates. Update package versions.
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 04523e4 on wehriam/test-updates into * on master*.

@wehriam
Copy link
Collaborator Author

wehriam commented Apr 25, 2016

@silentrob please review and merge.

@silentrob silentrob merged commit b21efad into master Apr 25, 2016
@silentrob
Copy link
Collaborator

Thanks for the PR @wehriam

@wehriam wehriam deleted the wehriam/test-updates branch April 26, 2016 17:50
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