-
-
Notifications
You must be signed in to change notification settings - Fork 192
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.
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
Register Drupal 6 core classes #154
Conversation
Thanks for this! I'm working to get the current test failures fixed before merging this in. |
Tests are now all green. As discussed on IRC though, can you add the feature file from jhedstrom/DrupalDriver#26 as a test here? |
Sure, let's give it a try |
Expanded the support for taxonomy terms and tested it ... I guess in order to make the tests passing here, we need to commit the branch in drupal-driver itself? On my local system they pass so far:
|
Can you temporarily update |
Ah, clever idea On Mon, Mar 23, 2015 at 7:52 PM, Jonathan Hedstrom <notifications@github.com
|
Tests are green again if you want to rebase this. |
Maybe it would be a good idea now to squash the commits into one? |
I've merged the DrupalDriver PR, so you can now switch |
Alright, let's "hop" its still green. |
Register Drupal 6 core classes
Awesome! Thanks! |
Register Drupal 6 core classes Signed-off-by: Jonathan Hedstrom <jhedstrom@gmail.com>
Fixes #153
This just registers the classes provided by the other PR