search

Gizra / Municipality

https://dev-municipality.pantheonsite.io/
1 stars 0 forks source link

Configure Travis [6h] #192

Closed amitaibu closed 7 years ago

amitaibu commented 7 years ago

Travis should support SimpleTest (instead of Behat that we will remove), and WDIO

AronNovak commented 7 years ago

@amitaibu This is in progress now, for the completion, we should to add our robot (https://github.com/gizra-robot) as a collaborator to the project, so he can post comments. Thank you!

AronNovak commented 7 years ago

@amitaibu price1 Also i'd need to have a write access for easier testing. The encrypted things might work in a fork, but not the other way around, if i encrypt the things in my fork, it would not work in another repository.

amitaibu commented 7 years ago

Granted admin right in this repo

AronNovak commented 7 years ago

@amitaibu Thanks. I am quite close to achieve the things, but I reached the timebox. Still need to do:

I'd say 2h is very likely enough

amitaibu commented 7 years ago

Bumped 2h.

amitaibu commented 7 years ago

instead of Behat that we will remove)

Lets keep the Behat for now.

amitaibu commented 7 years ago

Also, please add elm's unit tests

amitaibu commented 7 years ago

(The WDIO videos can be disabled for now if they are problematic.)

AronNovak commented 7 years ago

Behat tests are added again, Elm unit tests are added. Should we execute the things under PHP7? I think it should be the default for every new project, to make the performance gain available to our clients.

amitaibu commented 7 years ago

Sure, also lets add the setting to pantheon that converts env to php7 as-well