Skip to content
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

doc: add cargo test --doc --features __macro-docs near-sdk/src/near.rs to test job #1287

Closed
dj8yfo opened this issue Dec 28, 2024 · 0 comments
Labels

Comments

@dj8yfo
Copy link
Collaborator

dj8yfo commented Dec 28, 2024

for snippets in this module https://github.com/near/near-sdk-rs/blob/master/near-sdk/src/lib.rs#L54-L55 add
--features __macro-docs to test job , so they're tested and kept in sync.
currently they're correct:

: cargo test --doc --features __macro-docs near-sdk/src/near.rs

running 5 tests
test near-sdk/src/near.rs - near::private (line 70) ... ok
test near-sdk/src/near.rs - near::init (line 13) ... ok
test near-sdk/src/near.rs - near::handle_result (line 125) ... ok
test near-sdk/src/near.rs - near::payable (line 42) ... ok
test near-sdk/src/near.rs - near::result_serializer (line 97) ... ok
                                                                                
test result: ok. 5 passed; 0 failed; 0 ignored; 0 measured; 269 filtered out; finished in 1.00s

this is if these snippets are decided to be kept in near-sdk/src/lib.rs in scope of #1262 ,
and not moved to #[near] documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Shipped 🚀
Development

No branches or pull requests

1 participant