Skip to content

Misc#1611

Closed
DonJayamanne wants to merge 5 commits intocifrom
test
Closed

Misc#1611
DonJayamanne wants to merge 5 commits intocifrom
test

Conversation

@DonJayamanne
Copy link
Copy Markdown
Owner

@DonJayamanne DonJayamanne commented Sep 23, 2020

For #

  • 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.

Optional tests to run:
Select one or more of the following tests to run as part of the CI for the current PR. If multiple tests are selected and one of them fails, then the rest will not run.

  • Run single-workspace tests (with VS Code, without Python extension & without Jupyter)
  • Run functional-with-jupyter tests (mocked VS Code, without Python extension & with Jupyter)
  • Run integration tests (with VS Code, with Python extension & with Jupyter)

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.

1 participant