This issue is for any changes to be tried out on drupal.org testing infrastructure and gitlab pipelines. It is useful to have this issue open to be able to check what is happening on d.o. testing without adding confusing patches to other real issues.
Merge Request 72 (2.x) and Merge Request 115 (8.x-1.x)are marked as draft because it will never be committed. There is no need to rebase the MRs when other commits are made. When required the branch can be recreated and force pushed.
| Comment | File | Size | Author |
|---|
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
jonathan1055 commentedThis patch checks if PHPlint fails the syntax for a continue statement inside try/catch.
This did not fail in the scheduler patch on 192 here but the identical code did fail on the moderation integration patch in #2 here
Comment #4
jonathan1055 commentedThis patch in #2 was not the same. This new patch more closely replicates the change in moderation integration patch in #2 here
So this should fail the phplint checks
Comment #6
jonathan1055 commentedOK. So the phplint syntax checking is more stringent at php7.1 than at php5.6.
That's good to know.
Comment #7
jonathan1055 commentedSome more work has been done on #3251399: Drupalci.yml filtering for testgroups: 'scheduler,scheduler_api' does not run all the required tests and here is a test using alexpott's patch #9 from that issue instead of my patch #4
Comment #8
jonathan1055 commentedCheck if DevelGenerateTest can be run without
@group legacyComment #10
jonathan1055 commentedCheck @group legacy on 8.x-1.x
Comment #13
jonathan1055 commentedComment #15
jonathan1055 commentedOK, lets abandon that idea.
Comment #21
jonathan1055 commentedComment #25
jonathan1055 commentedComment #58
jonathan1055 commentedClosing this issue.