Skip to content

Raiden contracts 0.31.0#4665

Merged
pirapira merged 5 commits into
raiden-network:developfrom
pirapira:raiden-contracts-0.31.0
Aug 30, 2019
Merged

Raiden contracts 0.31.0#4665
pirapira merged 5 commits into
raiden-network:developfrom
pirapira:raiden-contracts-0.31.0

Conversation

@pirapira
Copy link
Copy Markdown
Contributor

@pirapira pirapira commented Aug 22, 2019

Description

This PR upgrades raiden-contracts dependency. It also contains some workarounds for raiden-network/scenario-player#230 (comment).

PR review check list

Quality check list that cannot be automatically verified.

  • Safety
  • Code quality
    • Error conditions are handled
    • Exceptions are propagated to the correct parent greenlet
    • Exceptions are correctly classified as recoverable or unrecoverable
  • Compatibility
    • State changes are forward compatible
    • Transport messages are backwards and forward compatible
  • Commits
    • Have good messages
    • Squashed unecessary commits
  • If it's a bug fix:
    • Regression test for the bug
      • Properly covers the bug
      • If an integration test is used, it could not be written as a unit test
  • Is it a user facing feature/bug fix?
    • Changelog entry has been added

@pirapira pirapira force-pushed the raiden-contracts-0.31.0 branch 9 times, most recently from 9e32ccb to ad311b5 Compare August 22, 2019 12:16
@pirapira pirapira force-pushed the raiden-contracts-0.31.0 branch from ad311b5 to 3ce9d6c Compare August 22, 2019 12:24
@pirapira
Copy link
Copy Markdown
Contributor Author

I've been struggling with raiden-network/scenario-player#230 (comment) (pinning many versions). But maybe it's easier to wait for #4646

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 28, 2019

Codecov Report

Merging #4665 into develop will decrease coverage by 0.16%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #4665      +/-   ##
===========================================
- Coverage    81.05%   80.89%   -0.17%     
===========================================
  Files          118      118              
  Lines        14226    14222       -4     
  Branches      2194     2193       -1     
===========================================
- Hits         11531    11505      -26     
- Misses        2052     2073      +21     
- Partials       643      644       +1
Impacted Files Coverage Δ
raiden/storage/serialization/types.py 59.09% <0%> (-9.1%) ⬇️
raiden/waiting.py 81.54% <0%> (-4.17%) ⬇️
raiden/tasks.py 68.11% <0%> (-2.18%) ⬇️
raiden/utils/__init__.py 81.13% <0%> (-1.57%) ⬇️
raiden/network/transport/matrix/client.py 68.5% <0%> (-1.3%) ⬇️
raiden/network/proxies/token_network.py 51.64% <0%> (-0.77%) ⬇️
raiden/raiden_event_handler.py 91.5% <0%> (-0.66%) ⬇️
raiden/api/python.py 67.72% <0%> (-0.53%) ⬇️
raiden/accounts.py 91.07% <0%> (-0.16%) ⬇️
raiden/network/transport/matrix/transport.py 81.28% <0%> (-0.15%) ⬇️
... and 7 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 0658a7c...7a10d2f. Read the comment docs.

Comment thread .circleci/config.yml Outdated
. ${VENV_PATH}/bin/activate
pip config set global.progress_bar off
pip install -U 'pip<19.0.0' wheel pip-tools
pip install -U 'pip==18.1' 'setuptools==40.8.0' 'wheel==0.33.6' 'pip-tools<3.9.0'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are these constraints still necessary with the cache change?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Probably not all of them. I'll try removing some of them (it's rather quick because the problem happens only a few minutes into the CI build.).

because the CI passes this way.

After this commit the line is the same as before this PR.
@pirapira pirapira force-pushed the raiden-contracts-0.31.0 branch from 5991619 to 7a10d2f Compare August 30, 2019 08:57
Copy link
Copy Markdown
Contributor

@palango palango left a comment

Choose a reason for hiding this comment

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

Looks better without these constraints.

Can you also add a comment to #4646 to remove this cache hack again?

@pirapira
Copy link
Copy Markdown
Contributor Author

Will do. Thanks.

@pirapira
Copy link
Copy Markdown
Contributor Author

Done #4646 (comment)

@pirapira pirapira merged commit 37b332b into raiden-network:develop Aug 30, 2019
@pirapira pirapira deleted the raiden-contracts-0.31.0 branch August 30, 2019 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants