Skip to content

Manual backport for PR 11622#11652

Closed
pllim wants to merge 1 commit into
astropy:v4.0.xfrom
pllim:pr11622-backport-test
Closed

Manual backport for PR 11622#11652
pllim wants to merge 1 commit into
astropy:v4.0.xfrom
pllim:pr11622-backport-test

Conversation

@pllim

@pllim pllim commented Apr 27, 2021

Copy link
Copy Markdown
Member

Description

This is a manual backport for #11622 . I am trying to see if the RTD job here points to "astropy-dev" as expected, since the doc here is not considered to be "dev".

I also noticed that git cherry-pick resulted in conflict (probably because LTS doc and dev doc have diverged so much), so this manual backport is unavoidable. But feel free to close without merge if the release manager finds otherwise.

DO NOT MERGE until after #11622 is merged.

TODO

  • Remove TMP commits after RTD builds.
  • Is Fix #11366 #11622 merged?
  • Does release manager need this or did auto-backport work?

@pllim

pllim commented Apr 27, 2021

Copy link
Copy Markdown
Member Author

Azure wheel failure is unrelated.

@pep8speaks

pep8speaks commented Apr 27, 2021

Copy link
Copy Markdown

Hello @pllim! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-04-30 16:26:43 UTC

Comment thread docs/conf.py Outdated
Comment thread docs/index.rst Outdated
Comment thread docs/install.rst

{%endif%}
``astropy``, such as an affiliated package, consider installing ``astropy``
into a :ref:`virtualenv <astropy-dev:virtual_envs>`.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@pllim pllim requested a review from embray April 27, 2021 23:52

@pllim pllim left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I can confirm that this renders for "stable" as expected. I will remove temporary commits and squash. Once I do that, RTD will fail again until #11466 is backported. Also, even if not, the rendering will be misleading because dev will resolve to True again until a release is cut.

tl;dr -- Just trust me.

Co-authored-by: Erik M. Bray <erik.m.bray@gmail.com>
@pllim pllim force-pushed the pr11622-backport-test branch from e16ba2d to 2b064c4 Compare April 30, 2021 16:26
@pllim pllim requested review from astrofrog and eteq April 30, 2021 16:27
@embray

embray commented May 17, 2021

Copy link
Copy Markdown
Member

It looks OK now?

@pllim

pllim commented May 17, 2021

Copy link
Copy Markdown
Member Author

It looks OK now?

I think so, but I'll leave it to the release manager to merge.

@astrofrog

Copy link
Copy Markdown
Member

Oops I backported this manually/locally in 0e44d7c so I think we can close this?

@pllim pllim closed this Aug 17, 2021
@pllim pllim deleted the pr11622-backport-test branch August 17, 2021 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants