Skip to content

fix: gnosis chain id in process request#1310

Merged
kernelwhisperer merged 1 commit intomainfrom
fix/process-request-gnosis
Jan 30, 2026
Merged

fix: gnosis chain id in process request#1310
kernelwhisperer merged 1 commit intomainfrom
fix/process-request-gnosis

Conversation

@kernelwhisperer
Copy link
Contributor

@kernelwhisperer kernelwhisperer commented Jan 30, 2026

Fixes https://github.com/cowprotocol/token-lists/actions/runs/21485728004/job/61894252009 triggered by #1308

Summary by CodeRabbit

  • Chores
    • Updated network configuration identifier naming for improved internal consistency and maintainability.

✏️ Tip: You can customize this high-level summary in your review settings.

@kernelwhisperer kernelwhisperer requested a review from a team January 30, 2026 08:56
@kernelwhisperer kernelwhisperer self-assigned this Jan 30, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

A configuration key in scripts/processRequest.mjs has been renamed from GNOSIS to GNOSIS_CHAIN within the NETWORK_CONFIG object, with all associated values and properties preserved.

Changes

Cohort / File(s) Summary
Configuration Key Rename
scripts/processRequest.mjs
Renamed GNOSIS to GNOSIS_CHAIN in the NETWORK_CONFIG object literal, preserving chainId and blockExplorer values.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A chain by any other name still sparkles bright,
From GNOSIS old to GNOSIS_CHAIN new—what a delight!
Configuration keys dance and rearrange,
While chainIds stay put through every change. 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: gnosis chain id in process request' accurately summarizes the main change—renaming GNOSIS to GNOSIS_CHAIN in the NETWORK_CONFIG and fixing a Gnosis chain ID issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/process-request-gnosis

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kernelwhisperer kernelwhisperer requested review from a team and removed request for a team January 30, 2026 10:00
@kernelwhisperer kernelwhisperer merged commit 1f130f3 into main Jan 30, 2026
6 checks passed
@kernelwhisperer kernelwhisperer deleted the fix/process-request-gnosis branch January 30, 2026 10:00
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2026
Copy link
Collaborator

@alfetopito alfetopito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙇 Thank you!!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants