LionsAd / drupal_ti

Drupal - Travis Integration
82 stars 37 forks source link

Support running tests based on 8.1.x. #70

Closed slashrsm closed 8 years ago

slashrsm commented 8 years ago

It seems that this is currently not possible.

Berdir commented 8 years ago

Yes please.

Berdir commented 8 years ago

By now, we might even want to make 8.1.x the default? d.o even defaults to 8.2.x now.

slashrsm commented 8 years ago

Yes! Updated default branch and rebased on top of master.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling bf047ede05c9fed804ef1cc91390f958410522e9 on slashrsm:allow_8_1_x into 9201ccd94dd9b85040e7b1742c6bd820fc9f3374 on LionsAd:master.

Berdir commented 8 years ago

See my referenced PR, this won't work without that, feel free to merge that into your PR and I'll close mine again

slashrsm commented 8 years ago

Done. #72 can be closed now.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling b2ce49cbeb59033e8b1170f80648e9114213f1a4 on slashrsm:allow_8_1_x into 9201ccd94dd9b85040e7b1742c6bd820fc9f3374 on LionsAd:master.

bojanz commented 8 years ago

+1 This is blocking Commerce at the moment.

Berdir commented 8 years ago

For anyone waiting on this, note that you can reference the commit here directly, see https://github.com/drupalcommerce/commerce/pull/361#issuecomment-203093729 for more information.

svendecabooter commented 8 years ago

+1 - this works for me, and should be default behaviour.

LionsAd commented 8 years ago

Merged thanks. Created follow-up issue to document the parameter.