github-workflow - Tidy up

maths · Mar 6, 2024 · 79b2cdb · 79b2cdb
1 parent 3bcb495
commit 79b2cdb
Show file tree

Hide file tree

Showing 9 changed files with 3 additions and 12 deletions.
diff --git a/.github/workflows/moodle-ci.yml b/.github/workflows/moodle-ci.yml
@@ -30,7 +30,6 @@ jobs:
     strategy:
       fail-fast: false
       matrix: # I don't know why, but mariadb is much slower, so mostly use pgsql.
-        # We use a mix of SBCL and GCL, but mostly prefer SBCL as it is faster.
         include:
           - php: '8.2'
             moodle-branch: 'master'
@@ -95,12 +94,12 @@ jobs:
         run: moodle-plugin-ci phpmd
 
       - name: Moodle Code Checker
-        continue-on-error: true # Currently fails. We really ought to get this passing.
+        continue-on-error: true
         if: ${{ always() }}
         run: moodle-plugin-ci codechecker --max-warnings 0
 
       - name: Moodle PHPDoc Checker
-        continue-on-error: true # Currently fails. We really ought to get this passing.
+        continue-on-error: true
         if: ${{ always() }}
         run: moodle-plugin-ci phpdoc
 

diff --git a/classes/cli_helper.php b/classes/cli_helper.php
@@ -589,7 +589,7 @@ public function create_gitignore(string $manifestpath):void {
     /**
      * Check if the repository has been initialised
      *
-     * @param string $manifestpath
+     * @param string $fullmanifestpath
      * @return void
      */
     public function check_repo_initialised(string $fullmanifestpath):void {

diff --git a/tests/cli_helper_test.php b/tests/cli_helper_test.php
@@ -49,7 +49,6 @@ public static function call_exit():void {
  *
  * Some tests are also done in import_repo_test where mocking has
  * already been set up.
- * @runTestsInSeparateProcesses
  *
  * @covers \gitsync\cli_helper::class
  */

diff --git a/tests/create_repo_test.php b/tests/create_repo_test.php
@@ -32,7 +32,6 @@
 /**
  * Test the CLI script for creating a repo from questions in Moodle.
  * @group qbank_gitsync
- * @runTestsInSeparateProcesses
  *
  * @covers \gitsync\create_repo::class
  */

diff --git a/tests/export_repo_test.php b/tests/export_repo_test.php
@@ -56,7 +56,6 @@ public static function handle_abort():void {
 /**
  * Test the CLI script for exporting a repo from Moodle.
  * @group qbank_gitsync
- * @runTestsInSeparateProcesses
  *
  * @covers \gitsync\export_repo::class
  */

diff --git a/tests/export_trait_test.php b/tests/export_trait_test.php
@@ -32,7 +32,6 @@
 /**
  * Test the CLI script for exporting a repo from Moodle.
  * @group qbank_gitsync
- * @runTestsInSeparateProcesses
  *
  * @covers \gitsync\export_repo::class
  */

diff --git a/tests/import_repo_test.php b/tests/import_repo_test.php
@@ -56,7 +56,6 @@ public static function handle_abort():void {
 /**
  * Test the CLI script for importing a repo to Moodle.
  * @group qbank_gitsync
- * @runTestsInSeparateProcesses
  *
  * @covers \gitsync\import_repo::class
  */

diff --git a/tests/lib_test.php b/tests/lib_test.php
@@ -35,7 +35,6 @@
 
 /**
  * Tests for library function in lib.php
- * @runTestsInSeparateProcesses
  * @group qbank_gitsync
  */
 class lib_test extends \advanced_testcase {

diff --git a/tests/tidy_trait_test.php b/tests/tidy_trait_test.php
@@ -33,8 +33,6 @@
  * Test the CLI script for exporting a repo from Moodle.
  * @group qbank_gitsync
  *
- * @runTestsInSeparateProcesses
- *
  * @covers \gitsync\export_repo::class
  */
 class tidy_trait_test extends advanced_testcase {