Skip to content

Conversation

@mathiasbynens
Copy link
Contributor

No description provided.

package.json Outdated
"version": "2.0.0",
"description": "jsPerf aims to provide an easy way to create and share test cases, comparing the performance of different JavaScript snippets by running benchmarks",
"description":
"jsPerf aims to provide an easy way to create and share test cases, comparing the performance of different JavaScript snippets by running benchmarks",
Copy link
Member

Choose a reason for hiding this comment

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

prettier may have nabbed this :/

"now": {
"alias": "jsperf.com"
"alias": "jsperf.com",
"env": {
Copy link
Member

Choose a reason for hiding this comment

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

thought this was easier to manage

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It totally is! Thanks

@maxbeatty
Copy link
Member

Hijacking this branch to debug this 😈 will you try to deploy this branch to now? should have more insight into why it isn't starting up with these changes

@mathiasbynens
Copy link
Contributor Author

Deploying…

@mathiasbynens
Copy link
Contributor Author

> ▲ npm start
> > jsperf.com@2.0.0 start /home/nowuser/src
> > node server.js
> 171212/210547.789, [log,debug] data: registering auth strategies
> 171212/210549.577, [log,info,db] data: executed 0 migrations
> 171212/210549.598, [log,info] data: connecting to redis cache
> No open port found. Make sure 'npm start' creates an HTTP server on a port (any).

server.expose('drop', client.drop.bind(client));

client.start(next);
client.start().then(next).catch(next);
Copy link
Member

Choose a reason for hiding this comment

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

catbox client start changed to being promise-based after v7 (oops) hopefully this solves the stall when connecting to rediscover

@mathiasbynens
Copy link
Contributor Author

That fixes it! ♥

@mathiasbynens mathiasbynens merged commit 3a1b91a into master Dec 12, 2017
@mathiasbynens mathiasbynens deleted the package-lock branch December 12, 2017 21:52
maxbeatty added a commit that referenced this pull request Oct 9, 2018
* master:
  Adding X-UA-Compatible in the head (#471)
  fix: .snyk & package.json to reduce vulnerabilities (#477)
  add link to wiki (#466)
  Fix issue with bad updates causing disappearing tests (#464)
  Fix cache start (#453)
  Ensure that edited test cases can be set as synchronous/asynchronous 
(#451)
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.

3 participants