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

riastores: Avoid deprecated --no-storage-sibling #641

Merged
merged 2 commits into from
Feb 2, 2021

Conversation

kyleam
Copy link
Contributor

@kyleam kyleam commented Jan 27, 2021

In the upcoming 0.14 release --no-storage-sibling is deprecated in
favor of --storage-sibling=off.


I don't recall where you landed on managing content for different DataLad releases. I'd imagine at the least you'll need to hold off on this a bit, but perhaps the text should be reworked to mention both options for compatibility with earlier releases.

In the upcoming 0.14 release --no-storage-sibling is deprecated in
favor of --storage-sibling=off.
@adswa
Copy link
Contributor

adswa commented Jan 27, 2021

Thanks a lot, @kyleam! :) I'll add a note on which version uses which option and merge this in once 0.14 is released.

unless :command:`create-sibling-ria` is run with the ``--storage-sibling=off``
flag. By default, it has the name of the RIA sibling, suffixed with ``-storage``,
The storage sibling is the git-annex ora-remote and is set up automatically --
unless :command:`create-sibling-ria` is run with ``--storage-sibling=off`` (in DataLad versions ``>0.14.``) or ``no-storage-sibling`` (in versions ``<0.14``).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Missing -- in front of no-storage-sibling?

Copy link
Contributor

Choose a reason for hiding this comment

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

! yes, thanks :)

@kyleam
Copy link
Contributor Author

kyleam commented Jan 27, 2021

I'll add a note on which version uses which option and merge this in once 0.14 is released.

Great, thanks for making my minimal change a proper update.

…options

Thanks to Kyle, this change introduces the successor of --no-storage-sibling and informs about
its deprecation. At the moment, both flags remain mentioned, but have a version identifier
attached.
In the same vein, because it fits, I added a quick note about the storage-sibling=only option
recently introduced.
@adswa
Copy link
Contributor

adswa commented Feb 2, 2021

Merging - 0.14 release is imminent

@adswa adswa merged commit 0eb3d6d into datalad-handbook:master Feb 2, 2021
@kyleam kyleam deleted the storage-sibling-off branch February 2, 2021 16:02
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