Skip to content

Add support for pytest 6 options#15095

Merged
kimadeline merged 1 commit into
microsoft:mainfrom
kimadeline:15094-pytest-6-options
Jan 11, 2021
Merged

Add support for pytest 6 options#15095
kimadeline merged 1 commit into
microsoft:mainfrom
kimadeline:15094-pytest-6-options

Conversation

@kimadeline
Copy link
Copy Markdown

@kimadeline kimadeline commented Jan 6, 2021

For #15094

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
  • Title summarizes what is changing.
  • Has a news entry file (remember to thank yourself!).
  • Appropriate comments and documentation strings in the code.
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated.
  • Test plan is updated as appropriate.
  • package-lock.json has been regenerated by running npm install (if dependencies have changed).
  • The wiki is updated with any design decisions/details.

@kimadeline kimadeline self-assigned this Jan 6, 2021
@kimadeline kimadeline marked this pull request as ready for review January 6, 2021 18:59
@kimadeline kimadeline added the skip tests Updates to tests unnecessary label Jan 11, 2021
ericsnowcurrently
ericsnowcurrently approved these changes Jan 11, 2021
View reviewed changes
Copy link
Copy Markdown

@ericsnowcurrently ericsnowcurrently left a comment

Choose a reason for hiding this comment

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

LGTM

@kimadeline kimadeline merged commit c5c613e into microsoft:main Jan 11, 2021
@kimadeline kimadeline deleted the 15094-pytest-6-options branch January 11, 2021 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@karthiknadig karthiknadig karthiknadig approved these changes

+1 more reviewer

@ericsnowcurrently ericsnowcurrently ericsnowcurrently approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@kimadeline kimadeline

Labels

skip tests Updates to tests unnecessary

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kimadeline @ericsnowcurrently @karthiknadig