Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Metal L2 chains #5094

Merged
merged 15 commits into from
May 18, 2024
Merged

Add Metal L2 chains #5094

merged 15 commits into from
May 18, 2024

Conversation

metallicusdev
Copy link
Contributor

Fixed the PR checks failing before

@metallicusdev
Copy link
Contributor Author

metallicusdev commented May 18, 2024

@ligi sorry for the bother, removed the extra line

Not clear on the icon error: https://cloudflare-ipfs.com/ipfs/QmQuz9BeubXcrN69Ri2b8vojRPknAbLdLnDDXqSsAcqs2f shows it to be svg

However the CI says expected svg, got txt

@ligi
Copy link
Member

ligi commented May 18, 2024

remove the whitespace in the first line of the svg

@metallicusdev
Copy link
Contributor Author

Thanks @ligi, updated and verified using exiftool

@ligi ligi merged commit 62987ba into ethereum-lists:master May 18, 2024
4 checks passed
Copy link

PR merged - please consider contributing some funds to lists.eth

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.

None yet

2 participants