-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Reorganize ResourceOp samples #1433
Reorganize ResourceOp samples #1433
Conversation
Hey @gaoning777, any opinion on this PR? |
Hey, Ilias, thanks for your contribution. I'm trying to collect some teamwide feedback in terms of the samples and sample test plans during the past two days. I will respond soon. Thanks |
/retest |
Hi there! Any update on this? Thanks! |
/lgtm |
/cc @gaoning777 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gaoning777 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Move the samples to different directories for the users to easier navigate to the desired usage demonstration.
Also add
README
s undersamples/volumes/
andsamples/snapshots/
with some documentation.This change is