Skip to content

l2geth: fix log error#507

Merged
gakonst merged 2 commits intomasterfrom
log/fix
Apr 20, 2021
Merged

l2geth: fix log error#507
gakonst merged 2 commits intomasterfrom
log/fix

Conversation

@tynes
Copy link
Copy Markdown
Contributor

@tynes tynes commented Apr 20, 2021

Description
Simple fix to a logger error where not enough arguments are passed. The logger expects a string followed by key value pairs. This PR adds the second argument to complete the key value pair.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 20, 2021

🦋 Changeset detected

Latest commit: 1789ce3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/l2geth Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Contributor

@gakonst gakonst 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.

@gakonst gakonst merged commit c4266fa into master Apr 20, 2021
@gakonst gakonst deleted the log/fix branch April 20, 2021 05:17
InoMurko referenced this pull request in omgnetwork/optimism May 25, 2021
* l2geth: fix log error

* l2geth: add changeset
bap2pecs added a commit to babylonlabs-io/optimism that referenced this pull request Jul 31, 2024
theochap pushed a commit that referenced this pull request Jan 15, 2026
I recommend removing the broken [Examples] section from both SUMMARY.md
and intro.md, as the path ./examples/README.md no longer exists in the
repository.
If there is a different location or an updated file that should replace
this link, feel free to suggest it, and I will update the references
accordingly.
emhane pushed a commit that referenced this pull request Feb 4, 2026
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