Skip to content

Remove error mapping in cosmology cloning - #11785

Merged
mhvk merged 1 commit into
astropy:mainfrom
nstarman:cosmo_deprecate_error_map
May 25, 2021
Merged

mhvk merged 1 commit into
astropy:mainfrom
nstarman:cosmo_deprecate_error_map

Conversation

@nstarman

@nstarman nstarman commented May 24, 2021 •

Copy link
Copy Markdown
Member

Signed-off-by: Nathaniel Starkman (@nstarman) [email protected]

A deprecation marked for v5.0.
Request review: @mhvk

@pllim

pllim commented May 24, 2021

Copy link
Copy Markdown
Member

I thought we tend to avoid starting a deprecation in a LTS release, no?

@nstarman

Copy link
Copy Markdown
Member Author

Sorry. I misspoke. This was already deprecated in v4.3 and is now removed.

@nstarman nstarman changed the title Deprecate error mapping in cosmology cloning Remove error mapping in cosmology cloning May 24, 2021
Comment thread docs/changes/cosmology/11785.api.rst Resolved
@nstarman
nstarman force-pushed the cosmo_deprecate_error_map branch from 562ee68 to 8530333 Compare May 24, 2021 18:30
Comment thread docs/changes/cosmology/11785.api.rst Resolved
@pllim pllim added this to the v5.0 milestone May 24, 2021
@pllim
pllim requested a review from mhvk May 24, 2021 18:37
@nstarman
nstarman force-pushed the cosmo_deprecate_error_map branch 2 times, most recently from d48f18e to 4b3770a Compare May 24, 2021 19:05

@mhvk mhvk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great, thanks!

@mhvk

mhvk commented May 25, 2021

Copy link
Copy Markdown
Contributor

But of course now it needs a rebase...

Co-authored-by: P. L. Lim <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
@nstarman
nstarman force-pushed the cosmo_deprecate_error_map branch from 4b3770a to c1ed121 Compare May 25, 2021 03:21
@mhvk

mhvk commented May 25, 2021

Copy link
Copy Markdown
Contributor

OK, all approved, rebased and green, so merging! Thanks, @nstarman

@mhvk
mhvk merged commit 04ee984 into astropy:main May 25, 2021
@nstarman
nstarman deleted the cosmo_deprecate_error_map branch May 25, 2021 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants