Skip to content

refactor: split the graph-node version fetch into steps - #239

Open
MoonBoi9001 wants to merge 1 commit into
mb9/drop-redundant-unicode-error-catchfrom
mb9/split-graph-node-version-fetch-into-steps
Open

MoonBoi9001 wants to merge 1 commit into
mb9/drop-redundant-unicode-error-catchfrom
mb9/split-graph-node-version-fetch-into-steps

Conversation

@MoonBoi9001

@MoonBoi9001 MoonBoi9001 commented Sep 10, 2026

Copy link
Copy Markdown
Member

This PR splits the code that asks 1 indexer for its graph-node version into 3 helpers (send the request, extract the version, decide whether to retry) with no intended change to what it returns.

@MoonBoi9001
MoonBoi9001 added this pull request to stack #242 September 10, 2026 13:51
@MoonBoi9001
MoonBoi9001 marked this pull request as ready for review September 10, 2026 13:54
Fetching one indexer's graph-node version had the HTTP call, the response shape check and
the retry decision all inside one loop body. Each is now a small helper, so the loop only
has to say: try once, stop on a 4xx, otherwise back off and try again.
@MoonBoi9001
MoonBoi9001 force-pushed the mb9/split-graph-node-version-fetch-into-steps branch from 87bcf34 to 2e3c874 Compare September 10, 2026 14:15
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.

1 participant