[Patternfly] Jekyll Version

[Andres Galante]

If its just a simple change, let just do a PR and make sure to write
release notes about it.

On Fri, Jul 15, 2016 at 12:15 PM, Jeffrey Phillips <jephilli at redhat.com>
wrote:

> I’m not are we need to schedule this as it is simply updating the
> README.md file, no other change is necessary.
>
>
> On Jul 15, 2016, at 8:30 AM, Andres Galante <agalante at redhat.com> wrote:
>
> Thanks Jeff for looking into this, great news :) It would be REALLY good
> if we can upgrade jekyll.
>
> Leslie can we schedule this one please?
>
> On Fri, Jul 15, 2016 at 9:26 AM, Jeffrey Phillips <jephilli at redhat.com>
> wrote:
>
>> Currently the patternfly README.md
>> <https://github.com/patternfly/patternfly/blob/master/README.md> file
>> states that jekyll 2.5.3 should be used when building due to Jekyll issues
>> #4433 <https://github.com/jekyll/jekyll/issues/4433> and #4442
>> <https://github.com/jekyll/jekyll/issues/4442>. These issues have been
>> resolved in Jekyll 3.1.4 and I believe we should remove the 2.5.3
>> requirement and update to the latest jekyll version. Not doing so causes
>> some issues since many folks miss this requirement causing a large number
>> of diffs to the dist/tests files. While the differences are non-functional,
>> it still causes many files to be changed and potentially committed.
>>
>> I’m reaching out for any objections or differing opinions before removing
>> this restriction from Patternfly. Coincidentally enough (not really) the
>> versions of the dist/tests files were built with Jekyll 3.1.6 so there
>> would be no changes to the dist/test files required.
>>
>> I have also logged issue #355 - Jekyll Version
>> <https://github.com/patternfly/patternfly/issues/355>.
>>
>>
>> _______________________________________________
>> Patternfly mailing list
>> Patternfly at redhat.com
>> https://www.redhat.com/mailman/listinfo/patternfly
>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/patternfly/attachments/20160715/ae48d80c/attachment.htm>