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

docs: fix docs of blockhash to reflect revert behaviour #3168

Merged
merged 1 commit into from
Dec 4, 2022

Conversation

ToonVanHove
Copy link
Contributor

What I did

Improved documentation of the blockhash built-in to describe its behaviour.
The blockhash function clamps its input and will thus revert if it is out of bounds. It will not return EMPTY_BYTES32 as was previously described.

How I did it

Updated the docs

How to verify it

See blockhash code

Commit message

docs: fix docs of blockhash to reflect revert behaviour

Description for the changelog

Update the documentation of the blockhash to accurately reflect its revert behaviour.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

Copy link
Member

@fubuloubu fubuloubu left a comment

Choose a reason for hiding this comment

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

Good catch!

@fubuloubu fubuloubu enabled auto-merge (squash) December 4, 2022 16:27
@codecov-commenter
Copy link

codecov-commenter commented Dec 4, 2022

Codecov Report

Merging #3168 (7f527b9) into master (046ea16) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3168      +/-   ##
==========================================
- Coverage   88.48%   88.40%   -0.08%     
==========================================
  Files          85       85              
  Lines       10756    10756              
  Branches     2286     2286              
==========================================
- Hits         9517     9509       -8     
- Misses        789      797       +8     
  Partials      450      450              
Impacted Files Coverage Δ
vyper/codegen/arithmetic.py 79.48% <0.00%> (-5.13%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@fubuloubu fubuloubu merged commit ac2fa14 into vyperlang:master Dec 4, 2022
@ToonVanHove ToonVanHove deleted the docs/fix-blockhash-docs branch December 6, 2022 16:52
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.

None yet

3 participants