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
global: addition of cli command for schema validation #4
Conversation
5e79939
to
4ccf2a9
Compare
def _warn_node(self, msg, *args, **kwargs): | ||
"""Do not warn on external images.""" | ||
if not msg.startswith('nonlocal image URI found:'): | ||
self._warnfunc(msg, '{0}:{1}'.format(get_source_line(node))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was removed by the previous sphinx fix.
a8131d0
to
37b5f62
Compare
|
||
By default the index of "array" "items" are ignored. Thus all the values of | ||
an array should have the same type in order to be compatible. | ||
This behavior can be disabled by setting in constructor: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
by setting "ignore_index=False" in the validator's constructor.
86b8474
to
3b3e284
Compare
# This file is part of DoMapping. | ||
# Copyright (C) 2015, 2016 CERN. | ||
# | ||
# DoMapping is free software; you can redistribute it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The project name is DoSchema, not DoMapping.
# -*- coding: utf-8 -*- | ||
# | ||
# This file is part of DoMapping. | ||
# Copyright (C) 2015, 2016 CERN. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The project name is DoSchema, not DoMapping.
This file is created in 2016, no need to add 2015 in the copyright.
031a796
to
dc74dfa
Compare
@@ -28,6 +28,16 @@ | |||
|
|||
import sphinx.environment | |||
|
|||
_warn_node_old = sphinx.environment.BuildEnvironment.warn_node |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you don't need it anymore. check latest cookiecutter-invenio-module (upgrade Sphinx).
9d0d959
to
3ed2c8d
Compare
37a0077
to
3ac7435
Compare
* NEW Adds a new command which validates schemas. Command comes with an '--ignore_indexes' option. Signed-off-by: PaulinaLach <paulina.malgorzata.lach@cern.ch>
@nharraud being part of @inveniosoftware/maintainers you should check commit message style before merging on Invenio software organization ( |
@jirikuncar Sorry for that. Next time I will also run kwalitee before merging. |
Command comes with an '--ignore_indexes' option.
Signed-off-by: Paulina1 paulina.malgorzata.lach@cern.cech