Skip to content

DOC: whats new other section reduced #40900

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 2 commits into from
Apr 13, 2021

Conversation

attack68
Copy link
Contributor

@@ -117,6 +117,9 @@ We provided some focused development on :class:`.Styler`, including altering met
to accept more universal CSS language for arguments, such as ``'color:red;'`` instead of
``[('color', 'red')]`` (:issue:`39564`). This is also added to the built-in methods
to allow custom CSS highlighting instead of default background coloring (:issue:`40242`).
Enhancements to other built-in methods include extending the :meth:`.Styler.background_gradient`
method to shade elements based on a given gradient map and not be restricted only to
values in the DataFrame (:issue:`39930`).
Copy link
Member

Choose a reason for hiding this comment

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

emm, just one nitpick, is 39930 an issue?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no thats the PR actually, which references the two issues: closes #22727 closes #28901. Is there a sphinx directive for PRs as opposed to issues? I wasn't previously aware of this.

Copy link
Member

Choose a reason for hiding this comment

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

nah, i don't think there is sphinx for prs, if so, would suggest to use (:issue:`22727`, :issue:`28901`), instead of (:issue:`39930`). Since usually put issue numbers to track in whatsnew, the 39930 is not issue but pr, then users who check it might get confused.

Copy link
Contributor

Choose a reason for hiding this comment

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

no they are the same (issue or pr), but yeah you can list all 3 (we usually list just the issues, however if the PR adds somehing non-trivial then can list that 2)

@jreback jreback added the Docs label Apr 12, 2021
@jreback jreback added this to the 1.3 milestone Apr 12, 2021
@jreback jreback merged commit 6cdf3e2 into pandas-dev:master Apr 13, 2021
@jreback
Copy link
Contributor

jreback commented Apr 13, 2021

thanks @attack68

@attack68 attack68 deleted the whats_new_section branch April 13, 2021 14:21
JulianWgs pushed a commit to JulianWgs/pandas that referenced this pull request Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: move *most* of Other Bug Fix section
3 participants