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

added-comment-on-functions #52

Merged
merged 2 commits into from
Feb 19, 2025
Merged

Conversation

tarikuamisganaw
Copy link
Contributor

I have added comments on the functions i did for better understanding

@YeabsiraNigusse
Copy link
Collaborator

@tarikuamisganaw your pull request has conflict with the main branch. try to pull the changes from the main branch and resolve the conflict.

@tarikuamisganaw
Copy link
Contributor Author

i have resolved the confilict and pushed it

@@ -19,6 +19,19 @@
; Function to insert an atom into the specifc bin
!(bind! &atombin (new-space))

;Function: insertAtom
Copy link
Contributor

Choose a reason for hiding this comment

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

Adding function names as a comment is not necessary as that information is already available on the function it self.

@samwondim
Copy link
Contributor

samwondim commented Feb 18, 2025

The comments seem to be too long. I wonder if there were ways you could cut down some of the lengthy ones. maybe try not commenting on functions whose actions are obvious to the point that one can understand what is happening in there only by reading the code.
Also I do not think the description of the type of the parameters and returned values is not important since it is already present in the type definition of the functions.

@tarikuamisganaw
Copy link
Contributor Author

I added comments on all functions because yeab asked to add comments on all the functions i did and the parameters and return type on the code are just types(kind of too generic),i just thought it needs more description

@YeabsiraNigusse
Copy link
Collaborator

The comments seem to be too long. I wonder if there were ways you could cut down some of the lengthy ones. maybe try not commenting on functions whose actions are obvious that one can understand what is happening in there only by reading the code. Also I do not thing the description of the type of the parameter and returned values is not important since it is already present in the type definition of the functions.

I tell them to document it the same way we write docstrings for Python functions. That's why the format looks like this.

Copy link
Collaborator

@YeabsiraNigusse YeabsiraNigusse left a comment

Choose a reason for hiding this comment

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

looks nice to me.

@YeabsiraNigusse YeabsiraNigusse merged commit 1398e4c into main Feb 19, 2025
1 check passed
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.

3 participants