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

Minor overhaul of our newsfragment categories #1593

Merged
merged 6 commits into from
Jun 9, 2020

Conversation

njsmith
Copy link
Member

@njsmith njsmith commented Jun 9, 2020

In particular, splits features into "regular features" and "headline
features", and split "removal" into "breaking changes without
deprecations" and "deprecations".

In particular, splits features into "regular features" and "headline
features", and split "removal" into "breaking changes without
deprecations" and "deprecations".
@codecov
Copy link

codecov bot commented Jun 9, 2020

Codecov Report

Merging #1593 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1593   +/-   ##
=======================================
  Coverage   99.69%   99.69%           
=======================================
  Files         109      109           
  Lines       13810    13810           
  Branches     1054     1054           
=======================================
  Hits        13768    13768           
  Misses         27       27           
  Partials       15       15           

pyproject.toml Outdated Show resolved Hide resolved
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks, I'll use deprecated to remove deprecated things in the future. By the way, should we mark the Read the Docs build as required?

Copy link
Member

@oremanj oremanj left a comment

Choose a reason for hiding this comment

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

1574.removal needs to be renamed to 1574.deprecated too, I think.

pyproject.toml Show resolved Hide resolved
@njsmith
Copy link
Member Author

njsmith commented Jun 9, 2020

Updated to address comments

I also marked the RTD build as required

(I guess we can get rid of the "Check docs" build?)

@njsmith
Copy link
Member Author

njsmith commented Jun 9, 2020

On further thought, I also rewrote some of the general instructions ins newsfragments/README.rst

@pquentin
Copy link
Member

pquentin commented Jun 9, 2020

(I guess we can get rid of the "Check docs" build?)

I guess we can! However the Read the Docs build just failed mysteriously, and I can't merge this since that build is now required. :)

@njsmith njsmith closed this Jun 9, 2020
@njsmith njsmith reopened this Jun 9, 2020
@njsmith
Copy link
Member Author

njsmith commented Jun 9, 2020

Huh, yeah, apparently the RTD build just.... stopped in the middle.

Well, all our CI systems sometimes fail due to random issues with the underlying infrastructure, so I guess this isn't a huge shock. I guess it's nice to know what that looks like on RTD :-)

@oremanj oremanj merged commit 919624f into python-trio:master Jun 9, 2020
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.

4 participants