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

[minor] Change retesteth port. #11

Merged
merged 2 commits into from
Sep 16, 2019
Merged

Conversation

shemnon
Copy link
Contributor

@shemnon shemnon commented Sep 16, 2019

PR description

Change the retesteth port to 47710 (0xba5e) so
(a) it does not conflict with other clients when started w/o args
(b) it is flavorful to the "Besu" name

Copy link
Contributor

@AbdelStark AbdelStark left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Change the retesteth port to 47710 (0xba5e) so
  (a) it does not conflict with other clients when started w/o args
  (b) it is flavorful to the "Besu" name

Signed-off-by: Danno Ferrin <[email protected]>
@shemnon shemnon merged commit 9aadbcb into hyperledger:master Sep 16, 2019
AbdelStark pushed a commit to AbdelStark/besu that referenced this pull request Sep 17, 2019
Change the retesteth port to 47710 (0xba5e) so
  (a) it does not conflict with other clients when started w/o args
  (b) it is flavorful to the "Besu" name

Signed-off-by: Danno Ferrin <[email protected]>
Signed-off-by: Abdelhamid Bakhta <[email protected]>
sambacha pushed a commit to blockarraygroup/besu that referenced this pull request Nov 22, 2019
shemnon referenced this pull request in shemnon/besu Oct 21, 2020
Signed-off-by: Danno Ferrin <[email protected]>
shemnon referenced this pull request in shemnon/besu Sep 17, 2021
Re-unify the gas calculators.  To avoid having to drag in a Transaction
class the gas calcualtor uses the relevant values from the transaction:
the payload size, if it is contract creation, and the access list
contents.

Signed-off-by: Danno Ferrin <[email protected]>
Add SPDX license header to docker_interop source files (#11)

Signed-off-by: Usman Saleem <[email protected]>
Signed-off-by: Danno Ferrin <[email protected]>
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.

2 participants