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

add system examination #954

Closed
wants to merge 3 commits into from
Closed

Conversation

ckote
Copy link
Collaborator

@ckote ckote commented Dec 3, 2024

No description provided.

@ojwanganto
Copy link
Contributor

To be honest, some of the commit messages are very general. Remember commit messages help when you want to roll back to a particular point in time and the rule of thumb is to clearly describe the exact change that has been introduced. For this particular PR, it would be nice to indicate the form so that one doesn't need to look at the changed files to get that answer

@andrineM
Copy link
Contributor

andrineM commented Dec 4, 2024

I agree with you @ojwanganto We also heavily rely on the commit messages when testing especially where there are not JIRA Tickets and for the release notes
@ckote @mmatheka @ErickSomie Let us please make the commit messages on forms more descriptive

@ckote ckote closed this Dec 10, 2024
@ckote ckote deleted the neurological_exam branch December 10, 2024 16:51
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.

4 participants