Skip to content

Conversation

@mkysel
Copy link
Collaborator

@mkysel mkysel commented Sep 26, 2025

This might need a better process lifecycle manager, but this is good enough now.

@mkysel mkysel requested a review from a team as a code owner September 26, 2025 15:28
@graphite-app
Copy link

graphite-app bot commented Sep 26, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • Queue - adds this PR to the back of the merge queue
  • Hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@macroscopeapp
Copy link

macroscopeapp bot commented Sep 26, 2025

Start the rates fetcher in cmd.replication.main.setupFeeCalculator to initialize the rate fetcher as part of starting the rate fetcher

In cmd/replication/main.go, the setupFeeCalculator helper creates a rates fetcher with fees.NewContractRatesFetcher, calls Start() on the returned fetcher, and returns an error if Start() fails before constructing and returning the FeeCalculator. See main.go.

📍Where to Start

Start with setupFeeCalculator in main.go, focusing on the creation of fees.NewContractRatesFetcher and the subsequent Start() invocation and error handling.


Macroscope summarized 04a593b.

@mkysel mkysel enabled auto-merge (squash) September 26, 2025 15:29
@mkysel mkysel merged commit 998c14a into main Sep 26, 2025
11 of 12 checks passed
@mkysel mkysel deleted the mkysel/poor-mans-rate-update branch September 26, 2025 15:57
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.

4 participants