-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update docs for 2.0.0-M2 and 2.0.0-M3 #2873
Update docs for 2.0.0-M2 and 2.0.0-M3 #2873
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2873 +/- ##
=======================================
Coverage 94.21% 94.21%
=======================================
Files 368 368
Lines 6948 6948
Branches 308 308
=======================================
Hits 6546 6546
Misses 402 402 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed that some of the authors are not correct. E.g. #2833. I will use the release note script to generate one and compare when I get some time next week
@kailuowang What do you mean? Do we not include doc / README editors here? I just ran |
I was referring to the authors mentioned in changes |
@travisbrown sorry I accidently just updated the PR directly, I meant to create a PR to your branch but github throws me a 404 when I tried. Then I clicked the wrong button... |
CHANGES.md
Outdated
* [#2827](https://github.com/typelevel/cats/pull/2827) In Apply.semigroup test replace ` ListWrapper` with `Option` by @kailuowang | ||
|
||
|
||
### 1 bug fixes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this is automated but can we at least fix the pluralization?
CHANGES.md
Outdated
* [#2821](https://github.com/typelevel/cats/pull/2821) removing unused resolver by @kailuowang | ||
|
||
|
||
### 2 test improvments |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And the spelling.
sort of related. #2881 is submitted to update the Process document. |
@LukaJCB will you have time to take another quick look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
The usual. That's a bunch of new contributors.
r? @rossabaker