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

Update FAQ #1324

Merged
merged 3 commits into from Dec 6, 2019
Merged

Update FAQ #1324

merged 3 commits into from Dec 6, 2019

Conversation

pirapira
Copy link
Contributor

@pirapira pirapira commented Oct 25, 2019

This closes #1219.

What this PR does

Updates the FAQ about two points:

  • the reason of existence of contracts_version has changed
  • now contracts_version bumps up to package_version roughly

Why I'm making this PR

The FAQ was outdated.

What's tricky about this PR (if any)

Nothing.


Any reviewer can check these:

  • Comment commits

And before "merge" all checkboxes have to be checked. If you find redundant points, remove them.

@codecov
Copy link

codecov bot commented Oct 25, 2019

Codecov Report

Merging #1324 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1324   +/-   ##
=======================================
  Coverage   81.77%   81.77%           
=======================================
  Files          21       21           
  Lines        1465     1465           
  Branches      193      193           
=======================================
  Hits         1198     1198           
  Misses        228      228           
  Partials       39       39
Impacted Files Coverage Δ
raiden_contracts/utils/versions.py 100% <ø> (ø) ⬆️

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 8d3d73f...710a583. Read the comment docs.

README.rst Outdated Show resolved Hide resolved
* the reason of existence of contracts_version has changed
* now contracts_version bumps up to package_version roughly

This closes raiden-network#1219.
So updating another entry in the FAQ.
Recently the CI started using a newer version of black,
so these changes are needed.
@karlb karlb merged commit 168c7e1 into raiden-network:master Dec 6, 2019
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.

Update why contracts_versions exist
3 participants