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

use pyyaml beta with security fix #156

Merged
merged 13 commits into from
Mar 28, 2019
Merged

use pyyaml beta with security fix #156

merged 13 commits into from
Mar 28, 2019

Conversation

fmigneault
Copy link
Collaborator

require beta version minimum of yaml until they fix and release 4.2/5.1 versions
yaml/pyyaml#243
yaml/pyyaml#193
yaml/pyyaml#74

@codecov-io
Copy link

codecov-io commented Mar 7, 2019

Codecov Report

Merging #156 into master will decrease coverage by 0.18%.
The diff coverage is 64.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #156      +/-   ##
==========================================
- Coverage    68.3%   68.12%   -0.19%     
==========================================
  Files          97       96       -1     
  Lines        6857     6889      +32     
==========================================
+ Hits         4684     4693       +9     
- Misses       2173     2196      +23
Impacted Files Coverage Δ
magpie/register.py 40.24% <0%> (-0.12%) ⬇️
magpie/adapter/magpieprocess.py 0% <0%> (ø) ⬆️
magpie/definitions/typedefs.py 0% <0%> (-10%) ⬇️
magpie/api/management/resource/resource_views.py 71.15% <0%> (-0.55%) ⬇️
.../438c27ec1c9_normalize_constraint_and_key_names.py 62.5% <100%> (ø) ⬆️
magpie/api/login/esgfopenid.py 63.15% <100%> (ø) ⬆️
magpie/api/home/home.py 100% <100%> (ø) ⬆️
magpie/definitions/pyramid_definitions.py 100% <100%> (ø) ⬆️
magpie/api/management/service/service_views.py 97.56% <100%> (ø) ⬆️
magpie/app.py 92.15% <100%> (ø)
... and 27 more

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 17a2386...6435886. Read the comment docs.

@fmigneault fmigneault added the security New security features or failing AuthN/AuthZ conditions label Mar 13, 2019
@fmigneault fmigneault merged commit 04858e0 into master Mar 28, 2019
@fmigneault fmigneault deleted the yaml-req branch March 28, 2019 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security New security features or failing AuthN/AuthZ conditions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants