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

Document the code #6

Open
alekay2200 opened this issue Jul 11, 2023 · 2 comments
Open

Document the code #6

alekay2200 opened this issue Jul 11, 2023 · 2 comments
Assignees
Labels
documentation Improvements or additions to documentation Sync Client Implementation the syncronous version of the websocket client
Milestone

Comments

@alekay2200
Copy link
Collaborator

Document all the code, docstrings and document to use rust doc

@alekay2200 alekay2200 added documentation Improvements or additions to documentation Sync Client Implementation the syncronous version of the websocket client labels Jul 11, 2023
@alekay2200 alekay2200 added this to the Syn Client milestone Jul 11, 2023
@alekay2200 alekay2200 self-assigned this Jul 11, 2023
@mfrandev
Copy link

Hi, I saw this project advertised on LinkedIn and want to contribute. If this issue isn't resolved yet, I'd like to do this task to gain familiarity with your implementation, then work on other issues.

@alekay2200
Copy link
Collaborator Author

Hi, thanks for be interested in my project, the issue is still open. Since the description of the issue can be not clear or misspelled I'll let you a few explanation.

The idea is to document each function and include a documentation test if the function is not complex enough, for instance:

/// This function returns the sum of to integers.
///
/// # Example
///
/// ```
/// let res = sum(3, 5);
/// assert_eq!(res, 8);
/// ```
fn sum(a: i32, b: i32) -> i32 {
    a + b
}

Also document each structure:

#[doc = "Represents a circle with a given radius."]
struct Circle {
    radius: f64,
}

Feel free to contact me if you have any doubts, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Sync Client Implementation the syncronous version of the websocket client
Projects
None yet
Development

No branches or pull requests

2 participants