Sitelet https://web.archive.org/web/20201203145219/https://github.com/huge-success/sanic/pull/1740
Skip to content
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

Release v19.12.0 #1740

Merged
merged 2 commits into from Dec 27, 2019
Merged

Release v19.12.0 #1740

merged 2 commits into from Dec 27, 2019

Conversation

@sjsadowski
Copy link
Contributor

@sjsadowski sjsadowski commented Dec 27, 2019

No description provided.

sjsadowski added 2 commits Dec 27, 2019
@sjsadowski sjsadowski requested a review from ahopkins Dec 27, 2019
@sjsadowski
Copy link
Contributor Author

@sjsadowski sjsadowski commented Dec 27, 2019

Just need to merge the changelog and version bump, then I'll cut the release

@codecov
Copy link

@codecov codecov bot commented Dec 27, 2019 •

Codecov Report

Merging #1740 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1740   +/-   ##
=======================================
  Coverage   92.18%   92.18%           
=======================================
  Files          22       22           
  Lines        2238     2238           
  Branches      419      419           
=======================================
  Hits         2063     2063           
  Misses        136      136           
  Partials       39       39
Impacted Files Coverage Δ
sanic/__version__.py 100% <100%> (ø) ⬆️

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 2b5f8d2...3411a12. Read the comment docs.

Copy link
Member

@seemethere seemethere left a comment

Just one comment about changing something in the pyproject.toml

Another note: your commit says that it’s updating from 19.12 to 19.12, you might want to edit it to reflect what it’s actually doing.

@@ -1,6 +1,6 @@
[tool.towncrier]
package = "sanic"
package_dir = "."
package_dir = ".."

This comment has been minimized.

@seemethere

seemethere Dec 27, 2019
Member

Was this change intentional? Not exactly versed on this particular file but didn’t really see a comment about it from the PR

This comment has been minimized.

@sjsadowski

sjsadowski Dec 27, 2019
Author Contributor

it was, sorry. @harshanarayana 's 'make release' was looking for the sanic package in the scripts/ dir, not in the root of the project, and towncrier was failing to execute.

@sjsadowski sjsadowski merged commit 075affe into master Dec 27, 2019
6 checks passed
6 checks passed
codecov/patch 100% of diff hit (target 92.18%)
Details
codecov/project 92.18% (+0%) compared to 2b5f8d2
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@sjsadowski sjsadowski deleted the release-v19.12.0 branch Dec 27, 2019
@sjsadowski sjsadowski restored the release-v19.12.0 branch Dec 27, 2019
@andreymal
Copy link
Contributor

@andreymal andreymal commented Dec 27, 2019

from markdown to restructured text

from RST to MD

😑

@sjsadowski
Copy link
Contributor Author

@sjsadowski sjsadowski commented Dec 27, 2019

@andreymal we're going to have to do some cleanup to get this one out the door anyway it looks like. I'll add that to the list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.