Skip to content

2.1 release #573

Closed
Closed

Description

@daspecster

I was wondering if it would be possible to cut the 2.1 release?

The https://github.com/PyCQA/pycodestyle/milestone/13 milestone appears to be complete.

If there's anything else that should be done before the release, I'm happy to help.

Activity

  1. IanLee1521 commented on Sep 15, 2016

    @IanLee1521
    Member

    Thanks for asking @daspecster ! Actually one thing that would be super helpful, and which I dropped the ball on is updating the changelog with everything that's happened since 2.0. Would that be something you would be willing to help with?

  2. daspecster commented on Sep 15, 2016

    @daspecster
    ContributorAuthor

    @IanLee1521, sure no problem!

  3. changed the title [-]2.1 release?[/-] [+]2.1 release[/+] on Oct 3, 2016
  4. IanLee1521 commented on Oct 3, 2016

    @IanLee1521
    Member

    @sigmavirus24 in particular, but anyone that wants to weigh in -- I was just looking over things for the 2.1 release, and I wanted to run something by you. One of the issues that was fixed was #530 which I believe may have had an unexpected side effect.

    By changing the name of the program (https://github.com/PyCQA/pycodestyle/blob/master/pycodestyle.py#L2050) from pep8 to pycodestyle this changes the config section name from pep8 to pycodestyle which would break (ignore) anyone's config that was using the old name. In the long run, that is what we want. But as a backwards incompatible change, I think it's an issue to address.

    I see two paths forward. First is to add logic that would handle either name for the section (around https://github.com/PyCQA/pycodestyle/blob/master/pycodestyle.py#L2148), but with a warning that the pep8 section name is deprecated in preference to pycodestyle. Or second, keep the change as is, but note it as backwards incompatible and release the next version as 3.0 instead of 2.1. This would also allow for merging in #433.

    Any thoughts (from anyone) on these two solutions?

  5. daspecster commented on Oct 3, 2016

    @daspecster
    ContributorAuthor

    My vote would be for option 2 with a message that the '[pep8]' section was deprecated.

    Supporting the ['pycodestyle] section is actually one of the changes that I'm looking for.
    googleapis/google-cloud-python#2020

  6. sigmavirus24 commented on Oct 3, 2016

    @sigmavirus24
    Member

    I would opt for option 1. But option 2 is suitable for me too.

  7. daspecster commented on Nov 4, 2016

    @daspecster
    ContributorAuthor

    @sigmavirus24 @IanLee1521, WDYT about cutting a release? Are there any other issues I could help with to get to 2.1 or 3.0?

  8. sigmavirus24 commented on Nov 4, 2016

    @sigmavirus24
    Member

    @IanLee1521 I'm happy to help by cutting the release.

  9. IanLee1521 commented on Nov 4, 2016

    @IanLee1521
    Member

    Done. Sorry for the ridiculous delay. That's on me.

    The new release has been cut and pushed to PyPI. Let me know if anyone notices any issues! Thanks for the help @daspecster ! And big thanks to @sigmavirus24 as always!

  10. daspecster commented on Nov 4, 2016

    @daspecster
    ContributorAuthor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions