Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

update requirements #4990

Merged
merged 2 commits into from
Dec 12, 2019
Merged

update requirements #4990

merged 2 commits into from
Dec 12, 2019

Conversation

etam
Copy link
Contributor

@etam etam commented Dec 12, 2019

I just used pip freeze on a fresh venv and manually adjusted multidict version. It has no further dependencies, so it should be fine.

Copy link
Contributor

@maaktweluit maaktweluit left a comment

Choose a reason for hiding this comment

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

python-dateutil==2.8.1

is now duplicate and should be removed from requirements-lint.txt
https://buildbot.golem.network/buildbot/#/builders/8/builds/16523

@shadeofblue
Copy link
Contributor

5 thumbs up

@codecov
Copy link

codecov bot commented Dec 12, 2019

Codecov Report

Merging #4990 into b0.22 will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            b0.22    #4990      +/-   ##
==========================================
- Coverage   90.11%   90.11%   -0.01%     
==========================================
  Files         233      233              
  Lines       21870    21870              
==========================================
- Hits        19709    19708       -1     
- Misses       2161     2162       +1

@shadeofblue shadeofblue merged commit 4a36ab7 into b0.22 Dec 12, 2019
@shadeofblue shadeofblue deleted the update_requirements branch December 12, 2019 14:53
@etam
Copy link
Contributor Author

etam commented Feb 7, 2020

by the way: multidict issue with pyinstaller should be fixed by now: pyinstaller/pyinstaller#4572

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants