Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove "Stable" from insider channels #6613

Closed
wants to merge 4 commits into from
Closed

Remove "Stable" from insider channels #6613

wants to merge 4 commits into from

Conversation

karrtikr
Copy link

For #2772

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

@karrtikr karrtikr added the no-changelog No news entry required label Jul 17, 2019
@karrtikr karrtikr force-pushed the modifyinsiders branch 2 times, most recently from 35451e1 to 10631c4 Compare July 17, 2019 18:32
@karrtikr karrtikr mentioned this pull request Jul 19, 2019
Closed
@DonJayamanne DonJayamanne self-requested a review July 25, 2019 18:48
@karrtikr karrtikr closed this Jul 25, 2019
@karrtikr karrtikr reopened this Jul 25, 2019
@karrtikr
Copy link
Author

Closing in favor of #6703

@karrtikr karrtikr closed this Jul 25, 2019
@karrtikr karrtikr deleted the modifyinsiders branch July 25, 2019 21:47
@lock lock bot locked as resolved and limited conversation to collaborators Aug 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DonJayamanne DonJayamanne Awaiting requested review from DonJayamanne

Assignees
No one assigned
Labels
no-changelog No news entry required
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@karrtikr