Skip to content

Input source 2.0.1 #102

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.

Sign up for GitHub

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

Jump to bottom
Open
fcollman wants to merge 85 commits into
base: dev
Choose a base branch
from
Open

Input source 2.0.1 #102

fcollman wants to merge 85 commits into from

Conversation

@fcollman
Copy link
Member

@fcollman fcollman commented Apr 13, 2020

No description provided.

@fcollman
flake8 and validate path
1eaea3d
@fcollman
flake8 warning cleanup
034cb74
@fcollman
bump to 2.0
4d9d68a
@fcollman
removing deprecated
afa3e30
@fcollman
flake8 improvements
5818c54
@fcollman
flake8 changes
a970d48
@fcollman
removing more deprecated items
0d35f2d
@fcollman
adding sources
63ed42e
@fcollman
fixed test so that input_json is valid json
00051f0
@fcollman
merging changes with sink options
8f06171
@fcollman
removed unused exception
56223cd
@fcollman
added pyyaml as requirement
c8a641a
@fcollman
removing unused __init__
a81bcc7
@fcollman
modified code to raise exception when the more than one configuration…
88afa43 
… is done, and considated looping code into private method
@fcollman
added exception to facilitate checking for extra configurations
72579ac
@fcollman
added more testing for yaml output source
da3cbc4
@fcollman
pep8
4dbd25c
@fcollman
fixed import error
e66b325
@fcollman
added tests for multiple input configurations
a0a0048
@fcollman
made it error if more than one output sink configured
a83a4cc
@fcollman
fixed bug in output_sink method
3e63c22
@fcollman
added 2 output config failure test
348812e
@fcollman
renaming default_configurable_sinks,sources
744d16f
@fcollman
doc typo
154e5ef
@fcollman
doc changes
8dca4a3
@fcollman
merging changes
84827b1
@fcollman
merging changes
619415e
@fcollman
fix yaml parser
6c133c4
@fcollman
removing pika source
d520847
@fcollman
made ConfigSource default __init__ and added auto imports to sources …
748233b 
…sub-package
fcollman and others added 28 commits February 22, 2019 07:24
@fcollman
fixed docstring
16e7c4b
@fcollman
wip
93a191f
@fcollman
alternative way of handling indent options
864d6f6
@fcollman
style changes
b18545b
@fcollman
flake8
4a418de
@fcollman
added marshmallow 3 compatability
f4da011
@fcollman
bump marshmallow req
ffaaae6
@fcollman
remove OptionList
ea82c89
@fcollman
fixing test
e129152
@fcollman
making alpha version
5774579
@fcollman
flake8
4de84bf
@fcollman
flake8
58eef15
@fcollman
flake8
1297faa
@fcollman
flake8
9963abc
@fcollman
python3 inpsect warning fix
537b03d
@fcollman
removing unused recursive checking
2054abc
@fcollman
bumping doc reqs
a5764a0
@fcollman
doc req fix
054aea1
@fcollman
merging master
5053f69
@NileGraddis
runtime input source selection
16232cb
@NileGraddis
tests for specific sources and sinks
ca0e76e
@NileGraddis
docstrings for config source; some renaming
1d414f3
@NileGraddis
avoid security vulnerability in yaml sour e
152212b
@NileGraddis
add example of multisource argschema parsing
ffdffc3
@NileGraddis
add missing docstring types; raise if url_source response is bad
9dafe5f
@NileGraddis
update url_source test class to handle raise_for_status
d58f2ef
@fcollman
Merge pull request #101 from NileGraddis/input_source_tweaks
d520533 
Input source tweaks
@fcollman
Merge branch 'dev' into input_source_2.0.1
69d5460
@codecov-io
Copy link

codecov-io commented Apr 13, 2020 •
edited
Loading

Codecov Report

Merging #102 into dev will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #102   +/-   ##
=======================================
  Coverage   93.63%   93.63%           
=======================================
  Files          16       16           
  Lines         644      644           
=======================================
  Hits          603      603           
  Misses         41       41

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 69d5460...69d5460. Read the comment docs.

@fcollman
Copy link
Member Author

fcollman commented Apr 28, 2020

this is superseded by #103

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fcollman @codecov-io @NileGraddis