-
Notifications
You must be signed in to change notification settings - Fork 37
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
Error running my first test #81
Comments
Hello I am currently checking if master still passes tests. It might be 8.2.x related. |
thanks. But anyhow, now i think it was i missunderstood the idea. In my repo i commited everything from core and my custom module too. And then, the script just does a git checkout from the drupal repo adding core files too and hence the error. Now i isolated my custom drupal module to their own repo, added the same travis.yml file and this error has gone away. |
And passing for me now!. https://travis-ci.org/Tantacom/tantaweb/jobs/134995635
Thanks a lot. great module btw. |
Oh, yes. This is how it should be. Just the module! :) |
In general it is a bad idea to commit core to your module. |
If you want to use your module with core, you should have another repo with core and import your module via composer.json. |
i see. we'll is not i was trying to commit core to my module, why i was doing was commiting core to my repo and then adding my custom module. is that what you meant? i was following, more or less this guide https://www.drupal.org/node/803746#code where "core" code is commited. is a guide for drupal 7 though |
Hello. I'm having a problem running the test. You can see the error and the entire output at
https://travis-ci.org/Tantacom/tantacom-web/jobs/134472696#L631
and my travis.yml file at https://github.com/Tantacom/tantacom-web/pull/5/files#diff-354f30a63fb0907d4ad57269548329e3R24
It looks like drupal is somehow installed two times... one under
/home/travis/build/Tantacom/tantacom-web/
and the other/home/travis/build/Tantacom/drupal-8/drupal/
any ideas of what could i be doing wrong?
The text was updated successfully, but these errors were encountered: