-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
DOC: update "What's new" with reference to cividis paper #12179
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
Conversation
I should probably add that the CircleCI builds are failing with an error that I saw myself when building, and when I said the patch compiles with sphinx I meant with the |
For this one I don't know what the best route of action is. I guess it makes sense to have it in the "what's new in 2.2.3" of the 2.2.3 docs, but of course in the "Previous what's new" of any subsequent release as well. Would that require two different PRs to be created, one against master and one against the 2.2.3 branch? |
Aren't the "Previously what's new" items just links to the generated what's news of earlier version? Is there a "What's new in 2.2.3" other than this one? |
Nevermind, you're right of course, found it. Since 2.2.3 is LTS for python 2 users we should edit both I believe. |
Submitted #12184 for master because it's easier to ask for forgiveness. |
The circle failures are understood, we have to backport a bunch of commits to this branch to cope with the sphinx 1.8 changes. |
@adeak the failures should be sorted now, would you be able to rebase this on to the current master branch? |
@dstansby I'm afraid I'm confused. Master should correspond to 3.0, so its |
I would have though it's enough to rerun the tests. I tried to trigger this by closing and reopening, but it won't work the circleci tests. |
99037f4
to
869a1c6
Compare
@ImportanceOfBeingErnest I touched the PR with a no-op commit amend; this seems to have triggered all the tests. |
No luck; though I'm not terribly surprised considering that the code base hasn't changed since I opened the PR. After Ernest's comment I was hoping there was some configuration change or something in the background that could have fixed the issue regardless. |
Ha, it's even worse, now travis fails as well for some unrelated reason. I think one has to wait until this gets fixed - those tests are beyond my paygrade. |
Unfortunately, CircleCI builds the head of the PR, not the merge of the PR and the target, like the other CIs do. That being said, there were no backports to the v2.2.3-doc branch, so it's unlikely for anything to work anyway. |
This should work now if rebased, for doc builds only. Travis fixes are still in progress. |
869a1c6
to
404b093
Compare
Thank you, all, indeed Circle works now and Travis breaks still. |
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 link works correctly: https://14327-1385122-gh.circle-artifacts.com/0/home/circleci/project/doc/build/html/users/whats_new.html
and as far as I understand, it's correct to directly merge this into v2.2.3-doc (see #12184 (comment)).
The Travis failure is fixed by #12366. Since this PR is trivial, I wouldn't insist on rebasing before merging.
Well, that fix has not come back to this branch anyway. |
Update the "What's new" for 2.2.3 with a reference to the now-published paper discussing cividis. I kept most of the original text (but added an accent to the name of the first author) and turned it into one long inline link pointing to the DOI of the paper published in PLoS ONE. The patched version compiles with sphinx.