Skip to content

Conversation

@rafal-ch
Copy link
Contributor

@rafal-ch rafal-ch commented Nov 28, 2025

Description

  • Add dht_provide() and dht_get_providers().
  • fn identity() functions returns identity, not just peer id

Related Issue(s)

This is one of the pieces needed for input-output-hk/hermes#630

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

@rafal-ch rafal-ch added the do not merge yet PR is not ready to be merged yet label Nov 28, 2025
@rafal-ch rafal-ch changed the title Implement dht_provide() and dht_get_providers() feat(hermes): Implement dht_provide() and dht_get_providers() Nov 28, 2025
@rafal-ch rafal-ch changed the title feat(hermes): Implement dht_provide() and dht_get_providers() feat(rust): Implement dht_provide() and dht_get_providers() Nov 28, 2025
@rafal-ch rafal-ch moved this from New to 👀 In review in Catalyst Nov 28, 2025
@github-actions
Copy link
Contributor

📚 Docs Preview

The docs for this PR can be previewed at the following URL:

https://docs.dev.projectcatalyst.io/libs/630_pre_publish_step

@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

Test Report | ${\color{lightgreen}Pass: 583/583}$ | ${\color{red}Fail: 0/583}$ |

@stevenj stevenj merged commit 0b9dabd into main Nov 28, 2025
25 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Catalyst Nov 28, 2025
@stevenj stevenj deleted the 630_pre_publish_step branch November 28, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge yet PR is not ready to be merged yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants