Repository navigation
Conversation
eaa4478 to
0bbcdcc
Compare
|
Thanks for the contribution! Couple friendly reminders:
Code-review wise I also notice you haven't added any new unit tests to validate the new functionality that's being added. Could you add those so we can make sure these detections continue to work if they need changes in the future? |
0bbcdcc to
7581895
Compare
Hi @MoralCode, Sorry... about the overlap! I had actually started working on this issue a while ago... but completely forgot to comment on the issue thread to claim it. I will definitely keep this in mind for next time to avoid duplicate PRs. I have updated the PR description to follow the pull request template and I've also added the required unit tests. Please review the changes and let me know if anything else is needed. Additionally, if there are any other high-priority issues that need attention.., please let me know- I'd be very happy to work on them!! |
Closes chaoss#112 Signed-off-by: Dev10-sys <kalpanagola9897@gmail.com>
7581895 to
13e4901
Compare
|
Thanks! Feel free to join the CHAOSS Slack in the #data-ai-disclosure-detection channel too if you'd like to interact with the other maintainers/contributors to this project or chat about the roadmap! I'll try and get around to this when I can |
suree!!! |
Description
noreply@codebuff.comandnoreply@freebuff.comtoKnownCoAuthorEmails.CodebuffandFreebufftoSupportedToolsInMentions.This PR fixes #112
Notes for Reviewers
Added unit test coverage across both
trailerandtoolmentionpackages to validate commit messages, Co-Authored-By headers, repo URLs, and PR text mentions.Signed commits
Generative AI disclosure
Please select one option: