search

d8-contrib-modules / cloudflare

Cloudflare Module for Drupal
16 stars 14 forks source link

Various cleanup #60

Closed phayes closed 6 years ago

phayes commented 7 years ago

See https://www.drupal.org/node/2913690

I've done a major cleanup to the 8.x branch, before committing, I'd like a review please.

To summarize the changes:

  1. Fixed the drupal composer repository
  2. Added a .gitignore
  3. Fixed all coder issues, the code is now Drupal coding-standards compliant.
  4. Fixed tests: A. syncs local directory to docroot/modules/program on every test run, not just on install B. Adds a run-tests.sh and composer test command to facilitate running tests easily C. Update tests so that they are now passing on drupal core 8.3, the current version of drupal D. Adds purge module as a dev dependancy (for testing) E. Adds instructions for testing into README.md
phayes commented 7 years ago

Working on fixing tests on travis now.

phayes commented 7 years ago

Now passes Travis and ready for review.