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

add troubleshooting section to readme; reiterate installing the latest voodoospark #34

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kevinold
Copy link

This might be just for me and my recent issue (#33) but I think this section and reiteration would help others (maybe just me in a few weeks/months).

@rwaldron
Copy link
Owner

I like it!

What else can we put in there? I'm about to go radio silent for 5 hours, but dig through the issues and see what else is good fodder for this and I can land it when I get to Seattle :)

@kevinold
Copy link
Author

I'll keep thinking on what tripped me up and that we can add to the list.

Another idea I had was to add a "postinstall" message to remind the user to
update voodoospark. I've never done it, but I think it'll work:

  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "postinstall": "echo \"Make sure you have the latest voodoospark!\""
  },

I can send this as another PR if you like the idea.

Kevin

Kevin Old
[email protected]

On Mon, Jul 28, 2014 at 4:01 PM, Rick Waldron [email protected]
wrote:

I like it!

What else can we put in there? I'm about to go radio silent for 5 hours,
but dig through the issues and see what else is good fodder for this and I
can land it when I get to Seattle :)


Reply to this email directly or view it on GitHub
#34 (comment).

@rwaldron
Copy link
Owner

Write a patch!

On Monday, July 28, 2014, Kevin Old [email protected] wrote:

I'll keep thinking on what tripped me up and that we can add to the list.

Another idea I had was to add a "postinstall" message to remind the user
to
update voodoospark. I've never done it, but I think it'll work:

"scripts": {
"test": "echo \"Error: no test specified\" && exit 1",
"postinstall": "echo \"Make sure you have the latest voodoospark!\""
},

I can send this as another PR if you like the idea.

Kevin

Kevin Old
[email protected] javascript:_e(%7B%7D,'cvml','[email protected]');

On Mon, Jul 28, 2014 at 4:01 PM, Rick Waldron <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');>
wrote:

I like it!

What else can we put in there? I'm about to go radio silent for 5 hours,
but dig through the issues and see what else is good fodder for this and
I
can land it when I get to Seattle :)


Reply to this email directly or view it on GitHub
#34 (comment).


Reply to this email directly or view it on GitHub
#34 (comment).

@kevinold
Copy link
Author

I have the message working with a "prepublish" or "postinstall" but the placement is the same and it gets printed before the dependencies are installed so it could get lost in the noise. Ideas?
screen shot 2014-07-29 at 10 17 37 am

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