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

Added tests for tables #114

Merged
merged 3 commits into from Jul 19, 2017
Merged

Added tests for tables #114

merged 3 commits into from Jul 19, 2017

Conversation

@shibasisp
Copy link
Contributor

@shibasisp shibasisp commented Jul 18, 2017

No description provided.

@pep8speaks
Copy link

@pep8speaks pep8speaks commented Jul 18, 2017

Hello @shibasisp! Thanks for updating the PR.

Line 65:5: E722 do not use bare except'
Line 73:5: E722 do not use bare except'

Comment last updated on July 18, 2017 at 21:29 Hours UTC
@coveralls
Copy link

@coveralls coveralls commented Jul 18, 2017

Coverage Status

Coverage increased (+7.9%) to 65.694% when pulling e9f361f on shibasisp:test_table_temp into 1fb25a8 on casacore:master.

@coveralls
Copy link

@coveralls coveralls commented Jul 18, 2017

Coverage Status

Coverage increased (+7.9%) to 65.694% when pulling 406a46a on shibasisp:test_table_temp into 1fb25a8 on casacore:master.

@tammojan tammojan merged commit cc4683c into casacore:master Jul 19, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+7.9%) to 65.694%
Details
@gijzelaerr gijzelaerr added this to the 2.2 milestone Nov 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants