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.

Sign up for GitHub

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

Jump to bottom

Implement Code Similarity Detection with jscpd Tool #3918

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Implement Code Similarity Detection with jscpd Tool #3918

wants to merge 1 commit into from

Conversation

Ducnguyen147
Copy link
Contributor

…at for integration with CodeChecker

@Ducnguyen147 Ducnguyen147 force-pushed the code_duplication branch 2 times, most recently from 76a0600 to 1b349d2 Compare May 22, 2023 16:44
@bruntib
Copy link
Contributor

bruntib commented Jun 6, 2023

Hi,
This is a great work, thank you!
I have just one usability question: the copy-paste codes are found properly and displayed in the CodeChecker interface. However, it shows only one message per function: "this is a copy-paste". But it doesn't display, where is the other code fragment which is the same. Could this be shown with a bug note?

@whisperity whisperity added analyzer 📈 Related to the analyze commands (analysis driver) other-analyzer 💬 Analysers which CodeChecker offers only report conversion of, but no driving labels Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bruntib bruntib Awaiting requested review from bruntib bruntib is a code owner

@vodorok vodorok Awaiting requested review from vodorok vodorok is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
analyzer 📈 Related to the analyze commands (analysis driver) new feature 👍 New feature request other-analyzer 💬 Analysers which CodeChecker offers only report conversion of, but no driving report-converter 🔄
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Ducnguyen147 @bruntib @whisperity @vodorok