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.

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

Add deprecation notice #71

Merged
merged 7 commits into from
Aug 24, 2023
Merged

Conversation

avishniakov
Copy link
Contributor

Added:

  • Repository deprecation after 01/08/2023 notice in README.md

@dagshub
Copy link

dagshub bot commented Jul 17, 2023

Copy link
Contributor

@fa9r fa9r left a comment

Choose a reason for hiding this comment

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

I think this needs a bit more info on why the repository will be deprecated, i.e., because the core repository will contain practical use cases in the examples/ folder which replace this repository

README.md Outdated Show resolved Hide resolved
@avishniakov avishniakov requested a review from fa9r July 17, 2023 12:09
README.md Outdated Show resolved Hide resolved
Co-authored-by: Felix Altenberger <[email protected]>
@avishniakov avishniakov marked this pull request as ready for review July 17, 2023 13:29
README.md Outdated
@@ -1,3 +1,10 @@
# IMPORTANT NOTE: THIS REPOSITORY IS GOING TO BE DEPRECATED AFTER 01/08/2023.
***Please consider the impact it might have on your ongoing work and plan accordingly.***
Copy link
Contributor

@fa9r fa9r Jul 17, 2023

Choose a reason for hiding this comment

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

Actually what's the purpose of this sentence? I guess users are not installing / building upon those repositories anyways so the deprecation shouldn't break anything, it only means that the examples are outdated / no longer maintaned.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it means that no maintenance and potentially not working code is here. Plus we will archive those, so they will get into a read-only state.
This sentence is more of a politeness - doesn't have to be a showstopper for any production work, but might get some tiny side-effects for Dev or Research.

@avishniakov avishniakov merged commit 2370f26 into main Aug 24, 2023
3 checks passed
@avishniakov avishniakov deleted the removal/FRW-2133-deprecation-warning branch August 24, 2023 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants