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.

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

Missing dev dependency on 'standard' command #508

Merged
merged 2 commits into from
Dec 1, 2016

Conversation

budda
Copy link
Contributor

@budda budda commented Nov 26, 2016

Please verify the following:

  • [ x] Everything works on iOS/Android
  • ignite-base ava tests pass
  • fireDrill.sh passed

Describe your PR

Git commits would fail with no standard command as it wasn't getting installed on project init.

Git commits would fail with no standard command as it wasn't getting installed on project init.
@@ -62,7 +62,8 @@
"reactotron-redux": "^1.5.2",
"reactotron-redux-saga": "^1.5.2",
"snazzy": "^5.0.0",
"standard-flow": "^1.0.0"
"standard-flow": "^1.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

did standard flow not work?

Copy link
Contributor Author

@budda budda Nov 26, 2016

Choose a reason for hiding this comment

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

Ignite default app gave me:

$ git commit -a -m "i like turtles"

> exampleapp@0.0.1 lint /Users/mike/Code/exampleapp
> standard --verbose | snazzy

sh: standard: command not found

I couldn't commit anything to Git until I installed 'standard'.

@budda
Copy link
Contributor Author

budda commented Dec 1, 2016

So no errors in the CI - other than ones outside this patches control regarding fsevents.

@GantMan
Copy link
Member

GantMan commented Dec 1, 2016

go ahead and merge master in for the CI, should be a button on github below this messsage.

@budda
Copy link
Contributor Author

budda commented Dec 1, 2016

@GantMan "Only those with write access to this repository can merge pull requests." is all I can see.

@GantMan GantMan merged commit 129c455 into infinitered:master Dec 1, 2016
@GantMan
Copy link
Member

GantMan commented Dec 1, 2016

thanks @budda! Welcome to the Ignite Contributor's list!

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.

2 participants