Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Switch over to docker. #43

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Switch over to docker. #43

wants to merge 4 commits into from

Conversation

jibran
Copy link
Collaborator

@jibran jibran commented Sep 14, 2017

  • Removes phing and replace it with Makefile.
  • Removes coder as it comes with core now.
  • Removes VM file

Please locally test this before merging.

jibran and others added 3 commits September 14, 2017 13:06
@pasankg
Copy link
Contributor

pasankg commented Jul 16, 2018

Instructions to setup

  • Clone the branch run composer install
  • Clone Drupal in app
  • Power up the containers
  • From inside the app dir run composer install
  • Run make install inside the php container

jibran++

@pasankg
Copy link
Contributor

pasankg commented Jul 30, 2018 •
edited
Loading

To run phpcs

cd app
export PATH="$PATH:/data/app/vendor/bin"
phpcs path/to/folder

To run tests

cd app
vendor/bin/phpunit -c core/phpunit.xml --filter=function_name path/to/the/Tests/FileTest.php

@jibran
Moar updates.
8b83d9e 
Signed-off-by: Jibran Ijaz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jibran @pasankg