Skip to content

cmd/geth, cmd/utils: rename config and flag to VMTraceJsonConfig#29573

Merged
holiman merged 1 commit intoethereum:masterfrom
ziogaschr:feat/rename-vmtracejsonconfig
Apr 24, 2024
Merged

cmd/geth, cmd/utils: rename config and flag to VMTraceJsonConfig#29573
holiman merged 1 commit intoethereum:masterfrom
ziogaschr:feat/rename-vmtracejsonconfig

Conversation

@ziogaschr
Copy link
Copy Markdown
Contributor

@ziogaschr ziogaschr commented Apr 18, 2024

This PR renames yaml config field VMTraceConfig to VMTraceJsonConfig, in order to be consistent with the renaming of the CLI flag.


In response to #29559 (comment), thanks for the nice catch @rjl493456442 and @s1na.

from VMTraceConfig to VMTraceJsonConfig
@holiman holiman changed the title Rename config and flag to VMTraceJsonConfig cmd/geth, cmd/utils: rename config and flag to VMTraceJsonConfig Apr 18, 2024
Copy link
Copy Markdown
Contributor

@holiman holiman 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
Copy Markdown
Contributor

@s1na s1na left a comment

Choose a reason for hiding this comment

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

LGTM

@s1na s1na added this to the 1.14.0 milestone Apr 23, 2024
@holiman holiman merged commit 882d1e2 into ethereum:master Apr 24, 2024
@ziogaschr ziogaschr deleted the feat/rename-vmtracejsonconfig branch April 24, 2024 10:46
jorgemmsilva pushed a commit to iotaledger/go-ethereum that referenced this pull request Jun 17, 2024
…thereum#29573)

renames the yaml config field VMTraceConfig to VMTraceJsonConfig, in order to be consistent with the renaming of the CLI flag.
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Sep 12, 2025
gzliudan added a commit to XinFinOrg/XDPoSChain that referenced this pull request Sep 13, 2025
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