Skip to content

CI rehearsal only (do not merge) - #32

Closed
michael-moffett wants to merge 2 commits into
mainfrom
surfpool-713-wbtc-row-a2-next
Closed

michael-moffett wants to merge 2 commits into
mainfrom
surfpool-713-wbtc-row-a2-next

Conversation

@michael-moffett

Copy link
Copy Markdown
Member

Fork CI only. Do not merge.

Join the Wrapped BTC row in `crates/types/src/verified_tokens.csv` back into one line, and add two tests that pin the row count and the WBTC mint.

The `5XZw…` icon URL has a newline inside its quotes. `csv.lines()` splits the row in two, `fields.len() < 5` skips both halves, and the mint is missing from `VERIFIED_TOKENS`.
@greptile-apps

greptile-apps Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] No code issue blocks merging, but the author explicitly says this CI rehearsal must not be merged.

Summary

Fixes an embedded newline in the WBTC catalog entry.

  • The verified-token catalog loads Wrapped BTC as WBTC.
  • Greptile uses the review settings listed in the new config.

Reviews (1) · Last reviewed commit: "greptile.json: review on request only" · Reviewed by Greptile

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