Repository navigation
Minor changes #164
Closed
drona-gyawali
started this conversation in
General
Replies: 1 comment
|
Hi @drona-gyawali, regaring the first point, I opened an issue: #176 Since the CONTRIBUTING file it's really important, I'll fix the link immediately, but thank you very much for your advise |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
While working and studying on the project, I noticed a couple of things that could be improved:
printstatements in the test cases. I believe these should be removed as they might cause confusion for new contributors and are not considered a good practice.https://github.com/certego/BuffaLogs/blob/develop/docs/CONTRIBUTING.mdshould be updated to:
https://github.com/certego/BuffaLogs/blob/develop/CONTRIBUTING.md.Before proceeding with a PR, I’d like to confirm if these fixes align with the project's guidelines. Please let me know your thoughts. Thank you.
All reactions