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

Fix typo in web3-utils.rst #3703

Merged
merged 5 commits into from
Aug 31, 2020
Merged

Conversation

eip1599
Copy link
Contributor

@eip1599 eip1599 commented Aug 25, 2020

Description

Fixes typos ("ethere unit") in web3-utils documentation.

Type of change

Fix typos. "ethere unit" => "ether unit"

Fixed typos ("ethere unit") in web3-utils documentation.
Copy link
Contributor

@spacesailor24 spacesailor24 left a comment

Choose a reason for hiding this comment

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

According to the Stack Overflow post (mentioned where these changes are being made), I believe the typo, ethere unit, should've been corrected to ether unit instead of ethereum unit

docs/web3-utils.rst Outdated Show resolved Hide resolved
docs/web3-utils.rst Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Aug 25, 2020

Coverage Status

Coverage decreased (-1.0%) to 84.328% when pulling 0b2ea3b on dev-xu:patch-eth-utils-typo into e13a50c on ethereum:1.x.

eip1599 and others added 2 commits August 26, 2020 22:30
Update to ether unit instead of ethereum unit

Co-authored-by: Wyatt Barnes <me@wyatt.email>
Update to ether unit instead of ethereum unit

Co-authored-by: Wyatt Barnes <me@wyatt.email>
Copy link
Contributor

@frankiebee frankiebee left a comment

Choose a reason for hiding this comment

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

thanks!

@frankiebee frankiebee merged commit 9edd908 into web3:1.x Aug 31, 2020
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.

4 participants