-
Notifications
You must be signed in to change notification settings - Fork 83
Add footnote that --schema is not supported in 5.24 #1870
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
Conversation
| This is done by using the `--verbose` option. | ||
| |import.report | ||
|
|
||
| |--schema=<path> footnote:[The optional `--schema` option is available in this version but not yet supported. It will be functional in a future release.] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I might be picky. But is it clear when we say 'the ... option is available but not supported'?
Could we just say that this option doesn't work with the incremental import command? Or - 'The incremental import command currently doesn't support the --schema option. It will be functional in a future release'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would be the best, but I didn't want to say that. I wanted to say it more nicely 😄. If we say, "The incremental import command currently doesn't support the—-schema option,", that is not true because we display it. I would say that incremental import supports it, but it doesn't work 😄 😄. It's a tricky one. I am open to more variations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what do you think about:
The --schema option is available in this version but not yet supported. It will be functional in a future release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we don't need the "optional" part.
Co-authored-by: NataliaIvakina <[email protected]>
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
Co-authored-by: NataliaIvakina <[email protected]>
Cherry-picked from #1870 Co-authored-by: NataliaIvakina <[email protected]>
No description provided.