Skip to content

Comments

Deprecate nonce_bump_journal_entry and caller_accounting_journal_entry functions#3367

Merged
rakita merged 1 commit intomainfrom
deprecate
Jan 28, 2026
Merged

Deprecate nonce_bump_journal_entry and caller_accounting_journal_entry functions#3367
rakita merged 1 commit intomainfrom
deprecate

Conversation

@rakita
Copy link
Member

@rakita rakita commented Jan 28, 2026

Summary

  • Deprecate nonce_bump_journal_entry and caller_accounting_journal_entry functions in JournalTr trait
  • Add #[deprecated] attributes to trait definitions and implementations
  • Add #[allow(deprecated)] to example implementations to suppress warnings

Test plan

  • Build passes
  • Clippy passes

Closes #3365

Deprecate `nonce_bump_journal_entry` and `caller_accounting_journal_entry`
functions in JournalTr trait and implementations.

Closes #3365
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 28, 2026

Merging this PR will not alter performance

✅ 173 untouched benchmarks


Comparing deprecate (3f86bfc) with main (f3b74d4)

Open in CodSpeed

@rakita rakita merged commit fc750ca into main Jan 28, 2026
31 checks passed
@rakita rakita deleted the deprecate branch January 28, 2026 18:34
@github-actions github-actions bot mentioned this pull request Jan 27, 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.

Deprecate nonce_bump_journal_entry(), caller_accounting_journal_entry

1 participant