Skip to content

Commit

Permalink
Remove phpcpd (deprecated) from dist and explain docs
Browse files Browse the repository at this point in the history
Better stop having the deprecated phpcpd command available
in the dist and explain docs, so newcomers won't enable it
by default. It's going to be removed in v5.

Part of moodlehq#262, credit goes to @jrchamp
  • Loading branch information
stronk7 committed Mar 16, 2024
1 parent 7b22048 commit 3cf6fee
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 15 deletions.
1 change: 0 additions & 1 deletion .travis.dist.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@ install:

script:
- moodle-plugin-ci phplint
- moodle-plugin-ci phpcpd
- moodle-plugin-ci phpmd
- moodle-plugin-ci phpcs --max-warnings 0
- moodle-plugin-ci phpdoc --max-warnings 0
Expand Down
5 changes: 0 additions & 5 deletions docs/GHAFileExplained.md
Original file line number Diff line number Diff line change
Expand Up @@ -142,11 +142,6 @@ jobs:
if: ${{ !cancelled() }} # prevents CI run stopping if step failed.
run: moodle-plugin-ci phplint

- name: PHP Copy/Paste Detector # DEPRECATED
continue-on-error: true # This step will show errors but will not fail
if: ${{ !cancelled() }}
run: moodle-plugin-ci phpcpd

- name: PHP Mess Detector
continue-on-error: true
if: ${{ !cancelled() }}
Expand Down
4 changes: 0 additions & 4 deletions docs/TravisFileExplained.md
Original file line number Diff line number Diff line change
Expand Up @@ -129,10 +129,6 @@ install:
script:
# This step lints your PHP files to check for syntax errors.
- moodle-plugin-ci phplint
# This step runs the PHP Copy/Paste Detector on your plugin.
# This helps to find code duplication.
# (DEPRECATED)
- moodle-plugin-ci phpcpd
# This step runs the PHP Mess Detector on your plugin. This helps to find
# potential problems with your code which can result in
# refactoring opportunities.
Expand Down
5 changes: 0 additions & 5 deletions gha.dist.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,11 +70,6 @@ jobs:
if: ${{ !cancelled() }}
run: moodle-plugin-ci phplint

- name: PHP Copy/Paste Detector
continue-on-error: true # This step will show errors but will not fail
if: ${{ !cancelled() }}
run: moodle-plugin-ci phpcpd

- name: PHP Mess Detector
continue-on-error: true # This step will show errors but will not fail
if: ${{ !cancelled() }}
Expand Down

0 comments on commit 3cf6fee

Please sign in to comment.