Skip to content

Conversation

@tkknight
Copy link
Contributor

@tkknight tkknight commented Nov 14, 2022

🚀 Pull Request

Description

Updated links and some surrounding text pointing to the Iris docs archive instead of the old docs rendered pages. The old docs rendered pages are planned to be deleted in favour of the archive (zip files).

This PR actions step 3 of #4396 (comment)

You can view the build docs here for the lifetime of this PR: https://tkknight-iris-test-doc.readthedocs.io/en/latest/index.html


Consult Iris pull request check list

@tkknight tkknight marked this pull request as ready for review November 14, 2022 12:13
Copy link
Contributor

@trexfeathers trexfeathers left a comment

Choose a reason for hiding this comment

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

Clear, and not verbose 👍

Searching your branch via an IDE confirms your original search - I think you've caught all instances referencing legacy documentation (tried phrases: legacy, 2.4, scitools.org.uk, and switch in case there was anything about switching versions).

@trexfeathers trexfeathers merged commit c0ab2e7 into SciTools:main Nov 14, 2022
trexfeathers pushed a commit to trexfeathers/iris that referenced this pull request Nov 16, 2022
* added link to the docs archive.

* added whatsnew
lbdreyer pushed a commit that referenced this pull request Nov 16, 2022
)

* Restore original as_data_frame behaviour, with new behaviour as opt-in.

* Document and test opt-in nature of mulit-d as_data_frame behaviour.

* Test fixes.

* Update for Pandas 1.5 deprecation.

* More explicit What's New.

* Fix int64 test.

* added link to the docs archive. (#5064)

* added link to the docs archive.

* added whatsnew

Co-authored-by: tkknight <2108488+tkknight@users.noreply.github.com>
@tkknight tkknight deleted the docs_archive branch November 9, 2023 15:42
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.

2 participants