pycodegen: Added graph property to LoadingOptions

[GlassOfWhiskey]

This property loads schemas in a Graph object. This feature can be
used for CWL file formats checking.

[lgtm-com]

This pull request introduces 2 alerts when merging ee04a03 into 9bf271d - view on LGTM.com

new alerts:

• 2 for Unused import

[codecov]

Codecov Report

Merging #583 (8af9d35) into main (2de4281) will decrease coverage by 0.01%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main     #583      +/-   ##
==========================================
- Coverage   83.08%   83.07%   -0.02%     
==========================================
  Files          20       20              
  Lines        3938     3947       +9     
  Branches     1100     1104       +4     
==========================================
+ Hits         3272     3279       +7     
- Misses        430      432       +2     
  Partials      236      236              

Impacted Files	Coverage Δ
schema_salad/fetcher.py	63.96% <80.00%> (+1.21%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[lgtm-com]

This pull request introduces 2 alerts when merging 42b4684 into 9bf271d - view on LGTM.com

new alerts:

• 2 for Unused import

[lgtm-com]

This pull request introduces 2 alerts when merging 4115757 into 9bf271d - view on LGTM.com

new alerts:

• 2 for Unused import

[mr-c]

Looking good! I'll wait to see tests here or in a cwl-utils PR that uses this

[mr-c]

schema_salad/fetcher.py:20: D101: Missing docstring in public class
schema_salad/fetcher.py:39: D101: Missing docstring in public class
schema_salad/fetcher.py:40: D107: Missing docstring in init
schema_salad/fetcher.py:44: D101: Missing docstring in public class
schema_salad/metaschema.py (100%)
schema_salad/python_codegen_support.py (100%)
schema_salad/tests/test_python_codegen.py (90.6%):
schema_salad/tests/test_python_codegen.py:101: D103: Missing docstring in public function
schema_salad/tests/test_python_codegen.py:112: D103: Missing docstring in public function
schema_salad/tests/test_python_codegen.py:120: D103: Missing docstring in public function

[GlassOfWhiskey]

schema_salad/fetcher.py:20: D101: Missing docstring in public class schema_salad/fetcher.py:39: D101: Missing docstring in public class schema_salad/fetcher.py:40: D107: Missing docstring in init schema_salad/fetcher.py:44: D101: Missing docstring in public class schema_salad/metaschema.py (100%) schema_salad/python_codegen_support.py (100%) schema_salad/tests/test_python_codegen.py (90.6%): schema_salad/tests/test_python_codegen.py:101: D103: Missing docstring in public function schema_salad/tests/test_python_codegen.py:112: D103: Missing docstring in public function schema_salad/tests/test_python_codegen.py:120: D103: Missing docstring in public function

Docstrings are missing everywhere in this repo. Do you want me to be the pioneer of Schema Salad documentators? XD

[GlassOfWhiskey]

BTW, I documented the tests. For the Fetcher classes, I'll leave the honors to someone of the developers. Also, since it is a hierarchy it would be nice to propagate docs from top to bottom or the other way round, but I don't know if some kind of Doc inheritance exists in Python

[GlassOfWhiskey]

It would be nice to propagate docs from top to bottom or the other way round, but I don't know if some kind of Doc inheritance exists in Python.

Starting from Python 3.5 docstrings are inherited when a subclass doesn't define a new one. Therefore, I only commented classes, constructors and the methods in the Fetcher class (all but the urljoin method, which is not totally clear to me).