-
Notifications
You must be signed in to change notification settings - Fork 58
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
Adding Azure instructions #237
Conversation
Signed-off-by: aice-support <ai.tce.support.baton@intel.com>
@srinarayan-srikanthan is there a way to include some ignores in the hyperlink tests? In particular for links that have an access token that we naturally cannot include to pass the failed test? |
@mkbhanda , yes looking into the fix for that. |
Further, that Azure home page is very generic .. taking the user through a lot of questions .. unless of course that is the only way. May be add "select 'skip'" |
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.
Let us be consistent, either use bold or double quotes of single quotes for commands/buttons/selection options. We currently have a mix.
Signed-off-by: aice-support <ai.tce.support.baton@intel.com>
Signed-off-by: aice-support <ai.tce.support.baton@intel.com>
We will handle this issue in CI. |
Signed-off-by: aice-support <ai.tce.support.baton@intel.com>
Signed-off-by: aice-support <ai.tce.support.baton@intel.com>
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.
done with all changes @mkbhanda
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.
2 minor edits suggested.
Signed-off-by: aice-support <ai.tce.support.baton@intel.com>
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.
LGTM
* added oci Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * modified links Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * added rag doc and png Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * fix broken link Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * standardization of code blocks Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * fix typo Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * rolling one commit back Signed-off-by: aice-support <ai.tce.support.baton@intel.com> * resolved suggestions from malini Signed-off-by: aice-support <ai.tce.support.baton@intel.com> --------- Signed-off-by: aice-support <ai.tce.support.baton@intel.com> Co-authored-by: aice-support <ai.tce.support.baton@intel.com> Co-authored-by: chen, suyue <suyue.chen@intel.com> Signed-off-by: alexsin368 <alex.sin@intel.com>
Added a tab for Azure and condensed two tabs to one.