Skip to content

Move splitParent from string.ts into tests folder#3989

Merged
DonJayamanne merged 1 commit intomicrosoft:masterfrom
DonJayamanne:moveString
Jan 18, 2019
Merged

Move splitParent from string.ts into tests folder#3989
DonJayamanne merged 1 commit intomicrosoft:masterfrom
DonJayamanne:moveString

Conversation

@DonJayamanne
Copy link

For #3988

  • 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!)
  • Unit tests & system/integration tests are added/updated
  • [n/a] Test plan is updated as appropriate
  • [n/a] package-lock.json has been regenerated by running npm install (if dependencies have changed)

Copy link

@d3r3kk d3r3kk left a comment

Choose a reason for hiding this comment

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

Looks fine, but why not keep the test? It's not truly necessary but it's there and no effort is needed...

@DonJayamanne DonJayamanne merged commit 43ef0ce into microsoft:master Jan 18, 2019
@DonJayamanne DonJayamanne deleted the moveString branch May 24, 2019 20:22
@lock lock bot locked as resolved and limited conversation to collaborators Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants