Skip to content

Conversation

adraffy
Copy link
Member

@adraffy adraffy commented Aug 17, 2025

  • added DNSRegistryResolver.sol and tests
    • supports resolve(multicall) if resolver does
    • uses IRegistryResolver.resolveWithRegistry()
    • ENS1 <this> <parentRegistry> <suffixName>
      • eg. sub.raffy.xyz + R xyzR.getSubregistry("raffy").getResolver("sub")

@immunefi-magnus
Copy link

immunefi-magnus bot commented Aug 17, 2025

🛡️ Immunefi PR Reviews

We’ve assigned 0 code reviewer(s) to this PR.

They’ll begin the review shortly and leave feedback directly in the pull request.

This review is based on the current state of your pull request. If you make changes after the review starts, they won’t be reflected here. To ensure the review includes your latest updates, you’ll need to open a new pull request.

@coveralls
Copy link

coveralls commented Aug 17, 2025

Pull Request Test Coverage Report for Build 17026015033

Details

  • 0 of 26 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.1%) to 63.089%

Changes Missing Coverage Covered Lines Changed/Added Lines %
contracts/src/L1/dns/DNSRegistryResolver.sol 0 26 0.0%
Totals Coverage Status
Change from base Build 17025409150: -1.1%
Covered Lines: 913
Relevant Lines: 1394

💛 - Coveralls

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.

2 participants