-
Notifications
You must be signed in to change notification settings - Fork 54
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 <[email protected]>
Signed-off-by: aice-support <[email protected]>
@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 <[email protected]>
Signed-off-by: aice-support <[email protected]>
We will handle this issue in CI. |
Signed-off-by: aice-support <[email protected]>
Signed-off-by: aice-support <[email protected]>
Signed-off-by: aice-support <[email protected]>
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 <[email protected]>
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 <[email protected]> * modified links Signed-off-by: aice-support <[email protected]> * added rag doc and png Signed-off-by: aice-support <[email protected]> * fix broken link Signed-off-by: aice-support <[email protected]> * standardization of code blocks Signed-off-by: aice-support <[email protected]> * fix typo Signed-off-by: aice-support <[email protected]> * rolling one commit back Signed-off-by: aice-support <[email protected]> * resolved suggestions from malini Signed-off-by: aice-support <[email protected]> --------- Signed-off-by: aice-support <[email protected]> Co-authored-by: aice-support <[email protected]> Co-authored-by: chen, suyue <[email protected]> Signed-off-by: alexsin368 <[email protected]>
Added a tab for Azure and condensed two tabs to one.