Skip to content
This repository was archived by the owner on Jan 2, 2019. It is now read-only.

Tried to fix travis configuration #704

Open
wants to merge 11 commits into
base: 1.8
Choose a base branch
from
Open

Tried to fix travis configuration #704

wants to merge 11 commits into from

Conversation

glady
Copy link

@glady glady commented Oct 16, 2015

Probem was:

  • All environments delivered: "No tests executed!" after phpcs call
  • Travis has new container-based environments, that can be used by "sudo: false"
    • faster starting
    • correct phpunit results
  • There was one test-file that used PHP5.4 array short syntax

Mike Gladysch added 8 commits October 16, 2015 19:25
…bug) and fixed short array notation for php 5.3 compatibility
…tations that look for "current year" (failing since 2014, increased 2 years (730 days)) - added comment for this test cases
… - expected by test was 1, not current month. alternative fix would be to expect current month
…ith with monday=1, which seems to be default), not a monday
… timeValue, function works another way!) and TIMEVALUE-data (different behavior of PHPExcel to MS Excel)
@glady
Copy link
Author

glady commented Oct 17, 2015

Ups. I just wanted to send the first commits with changing the travis-configuration because of their infrastructure changes.

The remaining commits containing some test provider comments, fixes and some trivial PSR-2 fixes.

There is also a fix within DateTime-Class (0 should be treated like NULL (failing sice change from May 2015) - defined by given test provider) - when the test provider is "out of date", this should be adapted again!

There were some trivial errors within provider-data. When there are tests, they should work correct or even give correct expected result.

Mike Gladysch added 3 commits November 22, 2015 14:56
…al decimals - only for case, that all remaining characters are # - not supported: "0.0#0# - is treated like "0.000#" (last test cases)
Merge PHPExcel into fork
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant