Skip to content

Conversation

michaelsproul
Copy link
Member

Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in eleel (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from self, and callers may want to compute block hashes without constructing an ExecutionLayer (eleel only constructs a simpler Engine struct).

@michaelsproul michaelsproul added ready-for-review The code is ready for review code-quality low-hanging-fruit Easy to resolve, get it before someone else does! v4.3.0 Estimated Q2 2023 labels May 24, 2023
@michaelsproul michaelsproul changed the base branch from stable to unstable May 24, 2023 04:14
Copy link
Member

@paulhauner paulhauner left a comment

Choose a reason for hiding this comment

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

LGTM!

@paulhauner paulhauner added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Jun 1, 2023
@paulhauner
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Jun 1, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 1, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 1, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 1, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 1, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 1, 2023

Build failed:

@paulhauner
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Jun 1, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 1, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 1, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 1, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 1, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Build failed:

@paulhauner
Copy link
Member

bors r-

@paulhauner
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Build failed:

@paulhauner
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@michaelsproul
Copy link
Member Author

bors r-
bors r+

@bors
Copy link

bors bot commented Jun 2, 2023

This PR was included in a batch that was canceled, it will be automatically retried

@bors
Copy link

bors bot commented Jun 2, 2023

Canceled.

bors bot pushed a commit that referenced this pull request Jun 2, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
@bors
Copy link

bors bot commented Jun 2, 2023

Pull request successfully merged into unstable.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title Expose execution block hash calculation [Merged by Bors] - Expose execution block hash calculation Jun 2, 2023
@bors bors bot closed this Jun 2, 2023
@michaelsproul michaelsproul deleted the exec-block-hash branch June 2, 2023 06:08
divagant-martian pushed a commit to divagant-martian/lighthouse that referenced this pull request Jun 7, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
ghost pushed a commit to oone-world/lighthouse that referenced this pull request Jul 13, 2023
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Proposed Changes

This is a light refactor of the execution layer's block hash calculation logic making it easier to use externally. e.g. in `eleel` (sigp/eleel#18).

A static method is preferable to a method because the calculation doesn't actually need any data from `self`, and callers may want to compute block hashes without constructing an `ExecutionLayer` (`eleel` only constructs a simpler `Engine` struct).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-quality low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge. v4.3.0 Estimated Q2 2023
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants