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

⬆️tooling-upgrades #2660

Merged
merged 9 commits into from
Nov 26, 2021
Merged

⬆️tooling-upgrades #2660

merged 9 commits into from
Nov 26, 2021

Conversation

GitHK
Copy link
Contributor

@GitHK GitHK commented Nov 26, 2021

What do these changes do?

Upgrades all the tools in all the packages.

respx is frozen to 0.17.1 until we upgrade it. this version is required by other packages and services.

From requirements/constraints.txt:

# respx>=0.18.0 require httpx>=0.21.0 which is frozen at 0.18.1 in most libraries
# remove this after upgrading httpx in all libraies
#   httpx==0.18.1 (from -c requirements/_base.txt (line 118))
#   httpx>=0.21.0 (from respx==0.19.0->-r requirements/_test.in (line 25))
respx==0.17.1

Related issue/s

How to test

Checklist

@GitHK GitHK self-assigned this Nov 26, 2021
@GitHK GitHK added this to the Meerkat milestone Nov 26, 2021
@codecov
Copy link

codecov bot commented Nov 26, 2021

Codecov Report

Merging #2660 (7dbe6aa) into master (edbd9e1) will increase coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2660   +/-   ##
======================================
  Coverage    78.0%   78.1%           
======================================
  Files         636     636           
  Lines       26060   26060           
  Branches     2523    2524    +1     
======================================
+ Hits        20351   20356    +5     
+ Misses       5042    5036    -6     
- Partials      667     668    +1     
Flag Coverage Δ
integrationtests 66.8% <ø> (-0.1%) ⬇️
unittests 73.6% <ø> (-0.1%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...simcore_service_webserver/projects/project_lock.py 94.7% <0.0%> (-5.3%) ⬇️
.../director/src/simcore_service_director/producer.py 61.0% <0.0%> (-0.5%) ⬇️
...tor_v2/modules/dynamic_sidecar/scheduler/events.py 95.0% <0.0%> (+2.1%) ⬆️
..._director_v2/modules/dynamic_sidecar/client_api.py 70.6% <0.0%> (+3.1%) ⬆️

@GitHK GitHK requested review from pcrespov and sanderegg November 26, 2021 07:20
Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

🎉 thx!

@pcrespov pcrespov merged commit 528f502 into ITISFoundation:master Nov 26, 2021
mrnicegyu11 pushed a commit to mrnicegyu11/osparc-simcore that referenced this pull request Dec 6, 2021
@sanderegg sanderegg added changelog:🔒️security t:maintenance Some planned maintenance work labels Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t:maintenance Some planned maintenance work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants