Skip to content

Conversation

@kernelwhisperer
Copy link
Contributor

@kernelwhisperer kernelwhisperer commented Dec 8, 2025

First commit updates the GH action, second commit manually updates the lists.

Testing

All new links resolve correctly.

The second commit was a simple find and replace of https://raw.githubusercontent.com/cowprotocol/token-lists/main/src/public with https://files.cow.fi/token-lists.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 8, 2025

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

138 files out of 246 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ 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/token-list-logo-urls

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 a review from a team December 8, 2025 11:48
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.

Tiny change needed to not break the image adding PR

@kernelwhisperer kernelwhisperer self-assigned this Dec 8, 2025
@alfetopito alfetopito merged commit d9dac3e into main Dec 8, 2025
6 checks passed
@alfetopito alfetopito deleted the fix/token-list-logo-urls branch December 8, 2025 16:04
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2025
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.

4 participants