Skip to content

Add policy regarding live test#6308

Merged
troydai merged 1 commit into
Azure:devfrom
troydai:doc
May 8, 2018
Merged

Add policy regarding live test#6308
troydai merged 1 commit into
Azure:devfrom
troydai:doc

Conversation

@troydai
Copy link
Copy Markdown
Contributor

@troydai troydai commented May 8, 2018

Also clean up the markdown lint errors.


This checklist is used to make sure that common guidelines for a pull request are followed.

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

  • I adhere to the Command Guidelines.

@troydai troydai requested a review from tjprescott May 8, 2018 21:00
@promptws
Copy link
Copy Markdown

promptws commented May 8, 2018

View a preview at https://prompt.ws/r/Azure/azure-cli/6308
This is an experimental preview for @microsoft users.

@troydai
Copy link
Copy Markdown
Contributor Author

troydai commented May 8, 2018

Will not wait for the CI since it is documentation change.

Copy link
Copy Markdown
Member

@tjprescott tjprescott left a comment

Choose a reason for hiding this comment

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

Couple typos but otherwise LGTM

Comment thread doc/authoring_tests.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This grammar is wonky. Do we even have any tests written in "nosetest form"? If not, we can just erase mention of it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I will remove it then.

Comment thread doc/authoring_tests.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Typo: "repeatly" > "repeatedly". "feature own" > "feature owner".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will fix.

Also clean up the markdown lint errors.
@troydai
Copy link
Copy Markdown
Contributor Author

troydai commented May 8, 2018

Updated.

@troydai troydai merged commit 63b1835 into Azure:dev May 8, 2018
@troydai troydai deleted the doc branch May 9, 2018 20:17
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.

3 participants