Skip to content

Conversation

@kclowes
Copy link
Collaborator

@kclowes kclowes commented Jun 16, 2022

What was wrong?

Added black checks to Beacon module, web3/scripts, and web3/gas_strategies

Related to Issue #2519

How was it fixed?

Added directories to tox file, ran black against correct directories and committed

Todo:

Cute Animal Picture

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

@kclowes kclowes changed the title Black beacon Add Black checks to beacon module, scripts/, and gas_strategies Jun 16, 2022
@kclowes kclowes requested review from fselmo and pacrob June 16, 2022 20:46
Copy link
Collaborator

@fselmo fselmo left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@pacrob pacrob left a comment

Choose a reason for hiding this comment

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

lgtm!

@kclowes kclowes merged commit f869679 into ethereum:master Jun 17, 2022
@kclowes kclowes deleted the black-beacon branch June 17, 2022 16:00
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.

3 participants