Skip to content

Conversation

bjlittle
Copy link
Member

🚀 Pull Request

Description

The PR marks the iris.util.as_compatible_shape function for deprecation from iris in a future release, in preference for using an instance of the iris.common.resolve.Resolve class.


Consult Iris pull request check list

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.

Thanks @bjlittle

As well as my comment below, there are a number of references within the test_util module. These should either be changed in this PR, or an issue should be raised for future work. Thanks!

@trexfeathers
Copy link
Contributor

there are a number of references within the test_util module. These should either be changed in this PR, or an issue should be raised for future work.

So it's apparently not appropriate to remove such references until the feature is actually removed, once it has been deprecated for long enough. There will be no need to raise an issue since final removal will cause test failures so they can't be missed.

@trexfeathers trexfeathers merged commit 50d5b42 into SciTools:master Sep 30, 2020
@bjlittle bjlittle deleted the deprecate-as-compatible-shape branch October 1, 2020 12:46
tkknight added a commit to tkknight/iris that referenced this pull request Oct 8, 2020
* upstream/master:
  add SciTools#3791 whatsnew entry (SciTools#3897)
  bump whatsnew latest and version to 3.1.dev0 (SciTools#3896)
  hide the further topics toc (SciTools#3894)
  Deprecate iris.util.as_compatible_shape (SciTools#3892)
  whatsnew additions (SciTools#3891)
  linkcheck ignore http://cfconventions.org (SciTools#3889)
  Cube arithmetic docs to master (SciTools#3890)
  added whats new for pr SciTools#3884 (SciTools#3887)
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.

2 participants