Skip to content

Conversation

@swang
Copy link
Contributor

@swang swang commented Mar 10, 2018

  • Use 4 spaces in Markdown to signify a sublist
  • Added extra instructions if you run into an npm install error
    involving python or if you are upgrading an older version of mysql and
    get an error complaining about the Column count

* Use 4 spaces in Markdown to signify a sublist
* Added extra instructions if you run into an `npm install` error
  involving python or if you are upgrading an older version of mysql and
  get an error complaining about the Column count
@maxbeatty
Copy link
Member

These are good debugging tips. I'm not sure the README is the best place for them (just to avoid any confusion). Maybe we can put them in a wiki?

@swang
Copy link
Contributor Author

swang commented Mar 12, 2018

Is there a jsperf wiki at the moment? I think sticking it in a wiki would be okay if it's actually used often otherwise it's not useful to anyone. The reason I stuck it the README is it makes it easier for developers to continue trying to setup a local development environment rather than have them stall during the process. Better to have it up front then having it being hidden somewhere and saves the developer a couple of trips to Google or Stackoverflow to find the issue.

@maxbeatty
Copy link
Member

I created a wiki and page for Local Development Debugging with the content from this PR. Want to add a link to it at the bottom of this section of the README?

@swang
Copy link
Contributor Author

swang commented Mar 14, 2018

Do you mean rewrite this PR to just add the link to the wiki? And what about changes to the markdown formatting? Right now in the README there's one line that just says, "install"

@maxbeatty maxbeatty mentioned this pull request Mar 14, 2018
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