Skip to content

Conversation

@jkrvivian
Copy link
Contributor

@jkrvivian jkrvivian commented Oct 30, 2024

Description of change

Remove the ParentSync trait, which only contains a deprecated method get_latest_parent_entry_ref_deprecated, and eliminate all its implementations.

Links to any relevant issues

Part of #2092

Type of change

  • Enhancement

Change checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have checked that new and existing unit tests pass locally with my changes

@jkrvivian jkrvivian added the node Issues related to the Core Node team label Oct 30, 2024
@jkrvivian jkrvivian self-assigned this Oct 30, 2024
@jkrvivian jkrvivian requested review from a team, miker83z and valeriyr as code owners October 30, 2024 14:08
@jkrvivian jkrvivian mentioned this pull request Oct 30, 2024
18 tasks
Copy link
Contributor

@miker83z miker83z 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! There is a leftover:

@muXxer muXxer requested a review from miker83z October 30, 2024 17:21
@bingyanglin bingyanglin merged commit acc9e45 into develop Oct 31, 2024
35 of 39 checks passed
@bingyanglin bingyanglin deleted the core-node/fix/remove-ParentSync branch October 31, 2024 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

node Issues related to the Core Node team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants