moodlehq/moodle-plugin-ci

Remove Copy/Paste Detector integration

stopfstedt opened this issue · 6 comments

stopfstedt commented

during composer install, i'm seeing this warning:

Package sebastian/phpcpd is abandoned, you should avoid using it. No replacement was suggested.

and true enough, the code repo has been flagged as abandoned. https://github.com/sebastianbergmann/phpcpd

i'm proposing the complete removal of this integration.

stronk7 commented

Hi @stopfstedt ,

yeah, we were aware about the removal. Also, we have been monitoring how forks (of phpcpd) are evolving, with a couple of them that "may" be interesting to switch to.

Thanks for creating this and #263, we'll analyse this soon, because one of the things we need to carefully consider is how the (potential) removal is going to affect to all the plugins out there that are running the phpcpd command, maybe requiring it to stay, but warning about the deprecation of the command, instead of a sudden deletion.

Let's see how everything evolves... I'll keep both this and the PR open as a reminder... hope we can come with a decision soon (switch, deprecate, remove...).

Ciao :-)

stopfstedt commented

@stronk7 thanks for the review.
i agree with your concerns - sudden removal might be a bit jarring.

i started with a new PR that deprecates this feature as an intermediary solution.

  • 👍1
stronk7 commented

Now that the deprecation has landed as part of #264 , I'll keep this open for whenever we make the next major version (5.0.0). Then, the phphpd command will be completely removed.

Ciao :-)

jrchamp commented

@stronk7 Now that phpcpd is deprecated, it seems like it should be removed from the sample workflow files. I understand that complete removal should happen in a major version (because it breaks backward compatibility), but the sample workflow files are more like "suggested best practices", which should probably be updated to stop using deprecated commands. With that said, whatever you decide, I'm fine either way.

  • 👍2
stronk7 commented

it seems like it should be removed from the sample workflow files.

Agreed, I've created #290 to remove it from dist and "explain" docs. Good idea!

Ciao :-)

stronk7 commented

Ok, I' think that this can be considered done.

The complete removal has happened as part of the https://github.com/moodlehq/moodle-plugin-ci/milestone/1 , and now it's part of the https://github.com/moodlehq/moodle-plugin-ci/tree/wip_v5 branch.

Ciao :-)

  • 👍1