Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

3.0-ify start/first-element/* #2539

Merged
merged 5 commits into from
May 1, 2018
Merged

3.0-ify start/first-element/* #2539

merged 5 commits into from
May 1, 2018

Conversation

ghost
Copy link

@ghost ghost commented Apr 26, 2018

@ghost ghost assigned arthurevans Apr 26, 2018
@ghost ghost added the 3.0 update label Apr 26, 2018
@ghost ghost requested review from keanulee, arthurevans and bicknellr April 26, 2018 06:54
@ghost ghost assigned bicknellr and keanulee Apr 26, 2018
yarn install

```
yarn install
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't use yarn anymore - use npm install and package-lock.json instead

@ghost
Copy link
Author

ghost commented Apr 29, 2018

@arthurevans ready for you to look at

@ghost ghost unassigned bicknellr and keanulee May 1, 2018
@ghost
Copy link
Author

ghost commented May 1, 2018

  • arthur review

Copy link

@arthurevans arthurevans left a comment

Choose a reason for hiding this comment

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

LGTM

@arthurevans
Copy link

I noticed one style thing I think we should do differently, but nothing that should block merging. I opened that as #2552.

@arthurevans arthurevans merged commit ad301cf into master May 1, 2018
@arthurevans arthurevans deleted the 3-0-first-element branch May 1, 2018 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3.0-ify start/first-element tutorial
4 participants