Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Rename portfolio recovery to building cluster recovery #559

Closed
navarroc opened this issue May 6, 2024 · 1 comment · Fixed by #569 or IN-CORE/incore-docs#385
Closed

Rename portfolio recovery to building cluster recovery #559

navarroc opened this issue May 6, 2024 · 1 comment · Fixed by #569 or IN-CORE/incore-docs#385
Assignees
@longshuicy
Milestone
1.19.0

Comments

@navarroc
Copy link
Member

navarroc commented May 6, 2024 •
edited
Loading

Rename Portfolio Recovery to Building Cluster Recovery analysis, but still allow using the old name with a deprecation warning (e.g. calling the old analysis by name should still work)
@navarroc navarroc added this to the 1.19.0 milestone May 6, 2024
@longshuicy longshuicy self-assigned this May 9, 2024
@longshuicy
Copy link
Member

This analysis is not used anywhere in the testbed notebooks. Based on our discussion in the dev meeting, can I safely just rename it without the deprecation process? @navarroc

This was linked to pull requests May 13, 2024
Rename portfolio to cluster recovery #569
Merged
update incore docs to reflect the renaming of portfolio recovery to cluster recovery IN-CORE/incore-docs#385
Merged
This was referenced Jun 6, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@longshuicy longshuicy

Labels
None yet
Projects
None yet
Milestone
1.19.0
2 participants
@navarroc @longshuicy