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

Polish News for release 1.0.0 #165

Merged
merged 3 commits into from
Jul 27, 2018
Merged

Polish News for release 1.0.0 #165

merged 3 commits into from
Jul 27, 2018

Conversation

dpseidel
Copy link
Collaborator

Reorganized bullets according to tidyverse style guide for news before release. Also polished for consistency throughout. Updated news/index.html with pkgdown.

@dpseidel
Copy link
Collaborator Author

You'll notice that I added links for the ggplot2 issues linked in news. although not quite kosher with the style guide this is to avoid pkgdown creating spurious links to scales issues automatically. Easily removed if you prefer.

@hadley hadley requested a review from batpigandme July 27, 2018 20:33
@hadley
Copy link
Member

hadley commented Jul 27, 2018

Overall form LGTM - @batpigandme can you please take a proof reading pass too?

Copy link
Contributor

@batpigandme batpigandme left a comment

Choose a reason for hiding this comment

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

Nice this looks great. Couple minor points, but overall 👍.

NEWS.md Outdated
* Remove `plyr` and `dichromat` dependencies. `dichromat` is now suggested
(@dpseidel, #118).
* `ordinal_format()` gains new arguments: `prefix`, `suffix`, `big.mark`, `rules`,
and rules for French and Spanish are also provided (@larmarange, #149).
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd switch a semi-colon in for the note at the end ⇨ "rules; rules for French and Spanish..."

NEWS.md Outdated

* `rescale_mid()` now properly handles NAs (@foo-bar-baz-qux, #104).
* `boxcox_trans()` is now invertible for `x >= 0` and requires positive values.
A new argument `offset` allows specification of both type 1 and type 2 Box-Cox
Copy link
Contributor

Choose a reason for hiding this comment

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

hyphens for "type-1" and "type-2" (since modifying …transformations)


# scales 0.3.0

* `rescale()` preserves missing values in input when the range of `x` is
(effectively) 0 (ggplot2#985).
(effectively) 0 ([ggplot2-985](https://www.github.com/tidyverse/ggplot2/issues/985)).

* Continuous colour palettes now use `colour_ramp()` instead of `colorRamp()`.
This only supports interpolation in Lab colour space, but is hundreds of
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a word missing here? "is hundreds of…" If no (which is totally possible), toss in a period?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

not sure why it's not showing up here but it does continue to say "hundreds of times faster." i've double checked the file.

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe you! The internet is weird.

@dpseidel dpseidel merged commit c197125 into r-lib:master Jul 27, 2018
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