<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, May 17, 2017 at 2:18 PM Floris Bruynooghe <<a href="mailto:flub@devork.be">flub@devork.be</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We have a deprecation cycle so we can't just break the API and bump to<br>
4.0 as I understand it.  So we accidentally broke the API, that sucks<br>
but generally I don't think we can fix that anymore</blockquote><div><br></div><div>That's true, we won't help anybody by bumping the version to 4.0 because that particular change (TerminalRepr being updated to new-style class) has already been in effect since 2016-12-05.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">given the deprecation cycle I think the only real option we have is to<br>
rollback all the other changes to subclass object as we know that will<br>
break more API instead of releasing that change. <br></blockquote><div><br></div><div>That's a very good point.<br></div><div><br></div><div><div dir="ltr">On Wed, May 17, 2017 at 5:35 PM Ronny Pfannschmidt <<a href="mailto:opensource@ronnypfannschmidt.de">opensource@ronnypfannschmidt.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Personally ID strongly prefer a major release because we do change an api that is consumed,<br></div></blockquote><div><br></div><div>You are right, but as Floris commented fortunately we can still rollback that change. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>IMHO we should do one on any case where we know beforehand,<br></div></blockquote><div><br></div><div>Definitely. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>My general impression is that the majority agrees on a minor release being sufficient, and I can go with that.</div></blockquote></div><div><br></div><div>Given all the points here I think we should rollback the PR which changed all classes to new-style classes, given that it might generate more breakages, and go for a 3.1 release. </div><div><br></div><div>The TerminalRepr change that went with 3.0.5 will remain as it is, as rolling back that at this point will generate more confusion.</div><div><br></div><div>I've opened a PR reverting the new-style classes change:</div><div><br></div><div><a href="https://github.com/pytest-dev/pytest/pull/2414">https://github.com/pytest-dev/pytest/pull/2414</a><br></div><div><br></div><div>Unless somebody has other points for discussion, I believe once we merge that PR we can go for the 3.1 release, what do you guys think?</div><div><br></div><div>Cheers,</div><div>Bruno.</div></div></div>