Skip to content

#254 part 1: Add changelog and unpin dev requirements #255

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 4, 2021

Conversation

j5awry
Copy link
Contributor

@j5awry j5awry commented Nov 30, 2020

Based on community feedback added back in a changelog. Done in markdown
to make copying into GH release notes easy. ALso unpinned dev
requirements, as pinning tox and black to much older versions for
development will cause issues in the near future.

@j5awry j5awry changed the title Add changelog and unpin dev requirements #254 part 1: Add changelog and unpin dev requirements Nov 30, 2020
@codecov
Copy link

codecov bot commented Nov 30, 2020

Codecov Report

Merging #255 (348f886) into master (3614028) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #255   +/-   ##
=======================================
  Coverage   96.86%   96.86%           
=======================================
  Files          19       19           
  Lines        2712     2712           
=======================================
  Hits         2627     2627           
  Misses         85       85           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3614028...af9229a. Read the comment docs.

ziirish
ziirish previously approved these changes Dec 9, 2020
Copy link
Contributor

@ziirish ziirish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome, thanks!

@j5awry
Copy link
Contributor Author

j5awry commented Dec 9, 2020

Oh shoot, I just saw that there's a check that the changelog is renderable by PyPi. I'm guessing this means that the PyPi automation includes pushing the changelog.

That means I'm going to need to convert this to .rst. BOOOO

@Colin-b
Copy link
Contributor

Colin-b commented Dec 9, 2020

How come pypi renders changelog ? (and pypi handles markdown syntax in project description since the new version of pypi, almost 2 years ago if I am correct)

Also, is there any particular reason this project would not follow https://keepachangelog.com/en/1.1.0/ ?

@ziirish
Copy link
Contributor

ziirish commented Dec 9, 2020

Automating the changelog generation is good when a good commit policy is in place. But since we almost exclusively rely on community patches, it's hard to enforce such policy. That's what we tried for a couple of releases, but it's too complicated in practice.

@j5awry I don't see why we should check the changelog complies with PyPi, maybe it's just a "bug" with our test suite.
I know we test the readme and that was justified a few years ago.

@j5awry
Copy link
Contributor Author

j5awry commented Dec 9, 2020

@Colin-b : never heard of keepachangelog till now. I'm using a slight derivation of a Debian changelog. PyPi rendering the changelog is actually nice. It's appended to the documentation so that when users drop by PyPi they see the changelog. You can see the example on flask-restplus.

As for the RST vs. MD, I'm just rolling with what we already had. It was cheeky of me to move to MD. just because I think it's easier to work in, doesn't mean it's the right choice. I should have consulted with everyone and checked all automation before just making the change. If we're moving to MD we should move everything to MD, and look at the automation.

@Colin-b
Copy link
Contributor

Colin-b commented Dec 9, 2020

@j5awry My bad, from what you said I supposed there was something dedicated to changelog on pypi while it is something done on this project to update the description of the project by appending the content of the changelog. If indeed you append documents, you should enforce the same formatting otherwise it will be hard to render ^^

As for changelog on pypi, you can add a changelog entry to the project link as it seems to be handled by pypi (you can see an example on this project). But if you'd rather put it at the end of the doc that's fine indeed, I am just not used to it ;)

Based on community feedback added back in a changelog. Done in markdown
to make copying into GH release notes easy. ALso unpinned dev
requirements, as pinning tox and black to much older versions for
development will cause issues in the near future.
Turns out there was a check for changelog compatibility with PyPi. The
changelog also looks to get published to PyPi. As such, converted the
.md to .rst using pandoc, then did a little cleaning up.
@j5awry j5awry force-pushed the prepare-0.3-release branch from 030e32b to af9229a Compare January 11, 2021 15:13
Copy link
Contributor

@ziirish ziirish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @j5awry!

@ziirish ziirish merged commit b31bd79 into python-restx:master Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants