Closed (fixed)
Project:
DrupalCI: Environments
Component:
PHP Containers
Priority:
Major
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
8 Sep 2015 at 14:53 UTC
Updated:
26 Jan 2017 at 04:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
neclimdulComment #3
neclimdulDon't really know how to test this but this seems like the appropriate change.
Comment #4
dawehnerWhat about running all core tests inside a patched test runner, just to be sure?
Comment #7
alan evans commentedI was just wondering why the YAML extension is only added here for 5.5 and 5.6 and the conclusion I came to was because 1) it only makes sense to add for versions that are supported on Drupal 8 and 2) Maybe there is no version of the extension available yet for PHP 7? Is that about right?
For future reference, it seems like some work has been done on it ( https://github.com/php/pecl-file_formats-yaml/tree/php7 ) but I doubt there's any easy package for install.
Oh and hi btw @neclimdul :) Long time no see ... hope you're well.
Comment #8
MixologicMostly #2. There doesnt yet exist a stable yaml extention for php7.
Comment #9
neclimdulYeah that about sums it up.
Hey Alan! Good to "see" you. ;)
Comment #11
yesct commentedAnd... now we need one without it also.
#2597762: Add a new testbot with (only) Symfony yaml parser (not the pecl one)
Comment #12
MixologicComment #13
elachlan commentedMoved to DrupalCI Environments.
Comment #14
MixologicThis now exists at the container level, and should be removed from the testrunner.