reth icon indicating copy to clipboard operation
reth copied to clipboard

Remove references to transition ids

Open petertdavies opened this issue 2 years ago • 3 comments

Describe the change

Transition ids were abolished and replaced with block numbers in #2275. However references to them still exist in docstrings and comments, which is quite confusing. For example the word "transition" appears 9 times in crates/storage/provider/src/post_state.rs.

Additional context

No response

petertdavies avatar May 03 '23 11:05 petertdavies

Yep - working on this next week.

gakonst avatar May 05 '23 22:05 gakonst

@onbjerg is this still the case?

mattsse avatar Jun 15 '23 15:06 mattsse

Not in that file specifically, but transition IDs are still mentioned in function names and docs, e.g. here:

https://github.com/paradigmxyz/reth/blob/600f3eac8c494dfc10b628527ff87513e2381b7f/crates/storage/provider/src/providers/database/provider.rs#L258

https://github.com/paradigmxyz/reth/blob/600f3eac8c494dfc10b628527ff87513e2381b7f/crates/storage/provider/src/providers/database/provider.rs#L123

https://github.com/paradigmxyz/reth/blob/600f3eac8c494dfc10b628527ff87513e2381b7f/crates/storage/provider/src/providers/database/provider.rs#L285

onbjerg avatar Jun 15 '23 15:06 onbjerg

This issue is stale because it has been open for 14 days with no activity.

github-actions[bot] avatar Aug 13 '23 01:08 github-actions[bot]

This issue is stale because it has been open for 14 days with no activity.

github-actions[bot] avatar Aug 29 '23 01:08 github-actions[bot]

This issue is stale because it has been open for 21 days with no activity.

github-actions[bot] avatar Sep 22 '23 01:09 github-actions[bot]

This issue is stale because it has been open for 21 days with no activity.

github-actions[bot] avatar Oct 17 '23 01:10 github-actions[bot]

This issue was closed because it has been inactive for 7 days since being marked as stale.

github-actions[bot] avatar Oct 25 '23 01:10 github-actions[bot]