-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Conversation
What has time changed that they don't run anymore? I don't get it. 3 Months ago the build passed: https://travis-ci.com/ecadius/jquery-mobile/builds/99162130 |
Consistent testing method which is described in the ReadMe.
use ubuntu 16.04
update php version
.travis.yml
Outdated
|
||
script: npm run-script ci | ||
script: npm run-script test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we don't need to change this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ReadMe describes the test process with npm run-script test
. What is the purpose of npm run-script ci
?
.travis.yml
Outdated
@@ -1,21 +1,21 @@ | |||
language: php | |||
|
|||
php: | |||
- "5.5" | |||
- "7.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC, @arschmitz used to say that there were some issues in upgrading to PHP 7.0.
Can you retrigger the build by pushing a commit? |
set php version back to 5.5
I reseted the php version back to 5.5. Perhaps the travis tests will pass, but they are a bit random and i simple try to rebuild my local environment which passed all tests. |
update npm
longer timeouts for the tests
Only one of the tests' suites is failing, we can figure that one later (has some issue related to phatomjs). Squashing and merging this one for now. Thanks. |
@apsdehal The issue at https://travis-ci.org/jquery/jquery-mobile/jobs/545864736 is not related to PhantomJS in particular but to the usage of |
The builds should run now. refs: #8639 #8618 #8617 #8245