Skip to content

feat(macros): support using #[dynify] on remote items#17

Merged
loichyan merged 7 commits into
mainfrom
feat/remote-item
Aug 30, 2025
Merged

feat(macros): support using #[dynify] on remote items#17
loichyan merged 7 commits into
mainfrom
feat/remote-item

Conversation

@loichyan
Copy link
Copy Markdown
Owner

Support using #[dynify] on remote items (either a trait or a function):

#[dynify(remote = "external_crate::Read")]
pub(crate) trait DynRead {
    async fn read_to_string(&mut self) -> String;
}

Note that the syntax for specifying arbitrary tokens as the value of an option of #[dynify] follows those used in serde.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 30, 2025

Codecov Report

❌ Patch coverage is 94.33962% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.09%. Comparing base (1474e09) to head (030f33a).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/lib.rs 0.00% 2 Missing ⚠️
macros/src/dynify.rs 98.03% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   98.37%   98.09%   -0.28%     
==========================================
  Files          12       13       +1     
  Lines         862      893      +31     
==========================================
+ Hits          848      876      +28     
- Misses         14       17       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@loichyan loichyan merged commit 16b3dc2 into main Aug 30, 2025
7 checks passed
@loichyan loichyan deleted the feat/remote-item branch August 30, 2025 07:29
@loichyan loichyan mentioned this pull request Sep 7, 2025
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.

1 participant