Skip to content

Conversation

@virkt25
Copy link
Contributor

@virkt25 virkt25 commented Mar 27, 2018

connected to #729

related discussion can be found at loopbackio/loopback.io#601

First Commit

  • Renamed example-getting-started to example-todo

Second Commit

  • Adds example-todo tutorial to side bar

Tests will FAIL till this PR lands as cli is not able to clone example-todo

Checklist

  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • API Documentation in code was updated
  • Documentation in /docs/site was updated
  • Affected artifact templates in packages/cli were updated
  • Affected example projects in packages/example-* were updated

@virkt25 virkt25 added this to the March 2018 milestone Mar 27, 2018
@virkt25 virkt25 self-assigned this Mar 27, 2018
@virkt25 virkt25 requested a review from a team March 27, 2018 21:31
@virkt25 virkt25 changed the title Rename/todo Garden path tutorial - rename getting-started to todo, add tutorial to docs Mar 27, 2018
virkt25 added a commit to loopbackio/loopback.io that referenced this pull request Mar 27, 2018
CODEOWNERS Outdated
packages/context/* @bajtos @raymondfeng
packages/core/* @bajtos @raymondfeng
packages/example-getting-started/* @bajtos
packages/example-todo/* @bajtos
Copy link
Member

Choose a reason for hiding this comment

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

Can we get at least one more additional CODEOWNER for this package?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm familiar enough with the package. I'll add myself.

BREAKING CHANGE: example-getting-started is now example-todo
@virkt25 virkt25 merged commit 7a09f1b into master Mar 29, 2018
@virkt25 virkt25 removed the review label Mar 29, 2018
virkt25 added a commit to loopbackio/loopback.io that referenced this pull request Mar 29, 2018
@virkt25 virkt25 deleted the rename/todo branch April 2, 2018 17:30
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.

6 participants