decipher/dcir

Drupal common CI Runner

Hello world

• I am a developer
• I use Drupal
• I use Nuxt
• I write Druxt

dcir's Issues

Add support for alternative versions of PHP

Allow Code review to fail without preventing simpletest to run.

Currently when Code review fails it prevents simpletest from running, it would be ideal if the tests still run, or at the least the behaviour be configurable.

Add Drupal Coder review test type.

Change Dockerfile source

Currently the Dockerfile uses drupalci/web-5.6, however there is a lot of stuff in that image that is not currently in use.

Add discovery logic and remove requirement on .dcir.yml file.

It would be ideal if when the .dcir.yml file isn't present that some discovery could be run to determine what type of project is being tested and what tests need to be run.

Error when dependency is a sub-module of an unavailable module.

When a dependency is a sub-module of an unavailable module, Entity Token for instance, DCIR will fail due to Drush failing.

Merge Dockerfile repository

As per Decipher/dcir-5.6#6

Add ability to execute specific runners

Currently DCIR will run Code review followed by Simpletest, it would be preferable if you could specify which runners you want in the .dcir.yml, as well as pass an argument into the docker run command to trigger an specific runner.

Don't error when no tests found

Currently with the automagic discovery, if there are no tests to run DCIR will error:

ERROR: No valid tests were specified.

PHPCS: Make verbose

If PHPCS doesn't find any issues it doesn't produce any output. Ideal if run as verbose.

Add support for Install profile projects.

Improved bash code styling

I suggest following https://google.github.io/styleguide/shell.xml#Variable_Names

Currently, global constants, env variables and one-time variables that can be local are using the same UPPERCASE format. Would be nice to have local vars as lowercase.

Also, lots of functionality is executed in global context without being split into functions. I suggest moving some functionality into functions to make contribution and debugging easy.

/root/.composer/vendor/bin/dcir: line 98: environment_discovery: command not found

PHPCS: Colourise output

As per Decipher/dcir-5.6#4

Add '--no-dependencies' flag

In the case of custom modules/features, the dependencies might not be accessible via drupal.org and therefore they can't be enabled.

Adding a '--no-dependencies' flag will allow DCIR to still be run regardless.

Add documentation

Replace PHPCS with DCR

To compensate for things like Features and other such code that shouldn't be considered as a Failure, PHPCS should be replaced with DCR.

Fix typos/mistakes in documentation.

So there's a few typos/mistakes the the documentation. One was more critical than the others and has been hotfixed, the other's can wait till 0.7.0 ideally.

Add Drush as a requirement.

Currently the DCIR Dockerfile is installing Drush, where in reality it should be a requirement of this package.

Add support for Drupal 7

Get (test) module dependencies.

Before running simpletest, DCIR needs to enable the module via Drush to ensure that any dependencies are available.

It will also need to ensure that any test dependencies are present.

Issue when info file contains multiple core entries due to package manager.

When a module has been packaged by the D.o package manager you end up with two core entries in the info file, one commented out and one injected.

As such, it prevents the simpletest runner to execute correctly.