[Patternfly] Jekyll Version

[Jeffrey Phillips]

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 <mailto: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 <mailto:Patternfly at redhat.com>
> https://www.redhat.com/mailman/listinfo/patternfly <https://www.redhat.com/mailman/listinfo/patternfly>
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/patternfly/attachments/20160715/4296015b/attachment.htm>