Skip to content

Silent data chains #7561

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ruisereno
Copy link

This PR adds the Silent Data chains.

Copy link

You successfully submitted a PR! Due to the amount of PRs coming in: we will only look at PRs that the CI is happy with. We can also not hold your hand getting the CI green - just look how others that where merged did it and RTFM. So as long as there is any CI check that reports an error - no human will look at this. You might be able to ask for some support after supporting the project - e.g. by sending funds to lists.eth. When you fixed things after a requested change - then you also need to (re-)request a review.

@ruisereno
Copy link
Author

The previously closed PR (#7363) has been pending for a while without being merged, and we've now resubmitted it. Thank you.

@campbell-codes
Copy link

campbell-codes commented Jul 30, 2025

Hi, @ligi @FrederikBolding.

We have made a donation to support the project: https://etherscan.io/tx/0x6a5308d4abcd0c03f3ada3343f77d6653757d043874dd62eef9fd2804c4061ef

The PR should pass one CI. Is there anything else we need to do to get this merged in?

If the PR falls behind we will be rebasing.

@FrederikBolding
Copy link
Collaborator

@ruisereno @campbell-codes CI is failing due to some invalid fields. Can take another look when fixed

@campbell-codes
Copy link

@FrederikBolding there was an issue with our shortname that we have now resolved. We've tried to run the checks locally and they now pass, we can't run on CI ourselves but we are quite confident it should work.

@campbell-codes
Copy link

Hi @FrederikBolding could you take another look at this one, we believe all issues have been resolved now.

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.

3 participants