Skip to content

fix: Remove the ping() placeholder from all three contracts - #82

Closed
VedantMadane wants to merge 1 commit into
FinesseStudioLab:mainfrom
VedantMadane:fix/issue-43
Closed

fix: Remove the ping() placeholder from all three contracts#82
VedantMadane wants to merge 1 commit into
FinesseStudioLab:mainfrom
VedantMadane:fix/issue-43

Conversation

@VedantMadane

Copy link
Copy Markdown

Summary

Remove the ping() placeholder from all three contracts

Changes

  • audit-registry\src\lib.rs
  • payment-router\src\lib.rs
  • usage-meter\src\lib.rs

Fixes #43

- audit-registry\src\lib.rs
- payment-router\src\lib.rs
- usage-meter\src\lib.rs

Fixes FinesseStudioLab#43

Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
@joelpeace48-cell

Copy link
Copy Markdown
Contributor

@VedantMadane ci fails, please fix failing ci

@joelpeace48-cell

Copy link
Copy Markdown
Contributor

@VedantMadane resolve conflicts and ensure ci passes

@VedantMadane

Copy link
Copy Markdown
Author

Closing as superseded: current main no longer has the stub ping() surface in audit-registry / payment-router / usage-meter (rewritten with real domain logic). Rebasing this branch onto tip leaves an empty/no-op diff, so there is nothing useful left to merge for #43.

Happy to reopen if maintainers still want a different follow-up.

@VedantMadane

Copy link
Copy Markdown
Author

Superseded: ping stubs already gone on main.

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.

Remove the ping() placeholder from all three contracts

2 participants