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

Validator function call bug fixes #187

Closed
wants to merge 1 commit into from
Closed

Validator function call bug fixes #187

wants to merge 1 commit into from

Conversation

oruebel
Copy link
Contributor

@oruebel oruebel commented Nov 2, 2017

Updated validator to use get_manager() instead of get_build_manager() and write_out() instead of write()

@oruebel oruebel requested a review from ajtritt November 2, 2017 21:13
@codecov-io
Copy link

codecov-io commented Nov 3, 2017

Codecov Report

Merging #187 into dev will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #187   +/-   ##
=======================================
  Coverage   57.69%   57.69%           
=======================================
  Files          43       43           
  Lines        4749     4749           
=======================================
  Hits         2740     2740           
  Misses       2009     2009
Flag Coverage Δ
#form 57.69% <0%> (ø) ⬆️
#integration 57.69% <0%> (ø) ⬆️
#pynwb 57.69% <0%> (ø) ⬆️
Impacted Files Coverage Δ
src/pynwb/validate.py 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc3f653...ff3aebe. Read the comment docs.

@@ -33,22 +33,22 @@ def print_errors(errors):
args = parser.parse_args()

if not os.path.exists(args.path):
write('%s not found' % path, file=sys.stderr)
write_out('%s not found' % path, file=sys.stderr)
Copy link
Collaborator

Choose a reason for hiding this comment

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

path -> args.path

No action required. I rebased the topic and will update it shortly.

@jcfr
Copy link
Collaborator

jcfr commented Nov 7, 2017

Closing. This PR is superseded by #204 and #205

@jcfr jcfr closed this Nov 7, 2017
@jcfr jcfr deleted the valbug branch November 7, 2017 19:26
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.

3 participants