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

feat: add documentation for functions in index and util files #65

Merged
merged 2 commits into from
Oct 30, 2024

Conversation

vickywane
Copy link
Contributor

@vickywane vickywane commented Oct 20, 2024

The aim of this PR is to create function documentation using the JsDocs format for functions in the index and utils files.

This PR will close #56

Copy link
Member

@Delta456 Delta456 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @vickywane! Thanks for opening PR. We appreciate your contribution. I have a few points which will make your PR even better

  • Do not elaborate too much on private functions
  • Functions documentation in general should be short and easy to understand
  • Examples are not needed for private functions.

@vickywane
Copy link
Contributor Author

Hi @Delta456

Thank you for the comment on this.

I will adjust my changes and request another review.

@vickywane
Copy link
Contributor Author

Hi @Delta456 I've reduced the elaborations on the function documentations and removed the examples for the private functions within the util.js file.

Looking forward to your next feedback.

@Delta456 Delta456 assigned Delta456 and harsha509 and unassigned Delta456 Oct 28, 2024
Copy link
Member

@harsha509 harsha509 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @vickywane !

@vickywane
Copy link
Contributor Author

You're welcome @harsha509 @Delta456 .

Looking forward to the merge 🫡!

@harsha509 harsha509 merged commit 78d8811 into LambdaTest:master Oct 30, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add documentation for each function
3 participants