<div dir="ltr">If its just a simple change, let just do a PR and make sure to write release notes about it.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 15, 2016 at 12:15 PM, Jeffrey Phillips <span dir="ltr"><<a href="mailto:jephilli@redhat.com" target="_blank">jephilli@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>I’m not are we need to schedule this as it is simply updating the README.md file, no other change is necessary.</div><div><div class="h5"><div><br></div><br><div><blockquote type="cite"><div>On Jul 15, 2016, at 8:30 AM, Andres Galante <<a href="mailto:agalante@redhat.com" target="_blank">agalante@redhat.com</a>> wrote:</div><br><div><div dir="ltr">Thanks Jeff for looking into this, great news :) It would be REALLY good if we can upgrade jekyll.<div><br></div><div>Leslie can we schedule this one please?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 15, 2016 at 9:26 AM, Jeffrey Phillips <span dir="ltr"><<a href="mailto:jephilli@redhat.com" target="_blank">jephilli@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Currently the patternfly <a href="https://github.com/patternfly/patternfly/blob/master/README.md" title="README.md" style="color:rgb(64,120,192);text-decoration:none;line-height:20px;white-space:nowrap;background-color:rgb(245,245,245)" target="_blank">README.md</a> file states that jekyll 2.5.3 should be used when building due to Jekyll issues <a href="https://github.com/jekyll/jekyll/issues/4433" style="color:rgb(64,120,192);text-decoration:none;line-height:24px;background-color:rgb(255,255,255)" target="_blank">#4433</a><span style="color:rgb(51,51,51);line-height:24px;background-color:rgb(255,255,255)"> and </span><a href="https://github.com/jekyll/jekyll/issues/4442" style="color:rgb(64,120,192);text-decoration:none;line-height:24px;background-color:rgb(255,255,255)" target="_blank">#4442</a>. 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.<div><br></div><div>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.</div><div><br></div><div>I have also logged issue #355 - <a href="https://github.com/patternfly/patternfly/issues/355" style="text-decoration:none;word-break:break-word;background-color:rgb(245,245,245)" target="_blank"><font color="#0056d6">Jekyll Version</font></a>.</div><div><br></div></div><br>_______________________________________________<br>
Patternfly mailing list<br>
<a href="mailto:Patternfly@redhat.com" target="_blank">Patternfly@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/patternfly" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/patternfly</a><br>
<br></blockquote></div><br></div>
</div></blockquote></div><br></div></div></div></blockquote></div><br></div>