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

Documentation: Using Comment blocks in functions #71

Closed
bms63 opened this issue Jul 6, 2022 · 5 comments · Fixed by #171
Closed

Documentation: Using Comment blocks in functions #71

bms63 opened this issue Jul 6, 2022 · 5 comments · Fixed by #171
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@bms63
Copy link
Collaborator

bms63 commented Jul 6, 2022

Please select a category the issue is focused on?

Function Documentation

Let us know where something needs a refresh or put your idea here!

It could be advantageous for developers to have commented blocks of code for future developers making updates. These comment blocks could point out salient features of the code and/or give a more detailed explanation of how the code works/processes. This would be seen by developers or by users inspecting the code on github.

We would like to establish a common way to use these comment blocks in our code.

DoD:
Programming strategy has recommended best practice for using comments blocks in code.

@sgorm123 Does this capture our discussion yesterday?

@bms63 bms63 added the documentation Improvements or additions to documentation label Jul 6, 2022
@bms63 bms63 transferred this issue from pharmaverse/admiral Sep 6, 2022
@bundfussr bundfussr self-assigned this Nov 15, 2022
@bundfussr
Copy link
Collaborator

@bms63 , @sgorm123 , I am not sure what you have discussed. I have added a "Comments" section in the programming strategy (see linked PR).
Could you have a look?

@bms63
Copy link
Collaborator Author

bms63 commented Nov 15, 2022

We never had a final decision on comment blocks. I will look over PR - thanks for taking this on.

@bundfussr
Copy link
Collaborator

@bms63 , should we remove the "release Q4" label? Or do you think we have an agreement until next week?

@bms63
Copy link
Collaborator Author

bms63 commented Nov 17, 2022

I think we can get this one resolved next week??

@bundfussr
Copy link
Collaborator

Maybe, let's try.

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
2 participants