apollo_l1_provider: rename catch_up_height and historic height

[guy-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[guy-starkware]

• apollo_l1_provider: replace bootstrapping with catching up #10726
• apollo_l1_provider: rename catch_up_height and historic height #10725 👈 (View in Graphite)
• apollo_l1_provider: scraper historic height is one more #10724
• apollo_l1_provider: minor updates to some bootstrapping tests of the L1 provider #10702
• apollo_l1_provider: add reset_bootstrapper function #10513
• apollo_l1_provider: check for uninitialized instead of bootstrapping+sync started #10512
• apollo_l1_provider: add verification that tx is pending when getting it from proposable index #10511
• apollo_l1_provider: improve add_tx and prevent double scraping #10508
• apollo_l1_provider: add l1_block_time as a config parameter #10505
• apollo_l1_provider: change L1_BLOCK_TIME to be 12 seconds #10544
• apollo_l1_provider: fix log message when skipping over historic heights #10465
• apollo_l1_provider: reduce number of provider error variants #10414
• apollo_l1_provider: remove some panics in the L1 Provider #10411
• main-v0.14.1

This stack of pull requests is managed by Graphite. Learn more about stacking.

[sirandreww-starkware]

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @guy-starkware and @ShahakShama)

---

crates/apollo_l1_provider/src/l1_provider.rs line 43 at r1 (raw file):

//   more blocks are committed while bootstrapping, they are applied after the target_height, and
//   the current_height will be set to one above the last block in the backlog.

Make this a doc comment using ///
optionally; add these docs to their fields

[ShahakShama]

@ShahakShama reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @sirandreww-starkware)

---

crates/apollo_l1_provider/src/l1_provider.rs line 43 at r1 (raw file):

Previously, sirandreww-starkware (Andrew Luka) wrote…

Make this a doc comment using ///
optionally; add these docs to their fields

+1

[guy-starkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @sirandreww-starkware)

---

crates/apollo_l1_provider/src/l1_provider.rs line 43 at r1 (raw file):

Previously, ShahakShama wrote…

+1

Only start height is a field of the Provider struct... these numbers are found in different places in the code. That's why I wanted to make sure they are all discussed in one place. I'll make it a docstring.

[guy-starkware]

@guy-starkware reviewed 1 of 4 files at r2, 1 of 1 files at r3, 3 of 3 files at r4, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @sirandreww-starkware)

[guy-starkware]

@guy-starkware dismissed @sirandreww-starkware from a discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @ShahakShama)

Approved by Shahak