Skip to content

feat: add recommended config eslint rule#14762

Merged
gaearon merged 2 commits into
react:masterfrom
SimenB:patch-2
Feb 28, 2020
Merged

feat: add recommended config eslint rule#14762
gaearon merged 2 commits into
react:masterfrom
SimenB:patch-2

Conversation

@SimenB

@SimenB SimenB commented Feb 6, 2019

Copy link
Copy Markdown
Contributor

Tested by copying in the built package in a project and following the added instructions in the readme

@sizebot

sizebot commented Feb 6, 2019

Copy link
Copy Markdown
Fails
🚫

node` failed.

Log

Error:  { FetchError: invalid json response body at http://react.zpao.com/builds/master/_commits/d827445a500e497712a1a404c7ccc25779b1df5c/results.json reason: Unexpected token < in JSON at position 0
    at /home/circleci/project/node_modules/node-fetch/lib/body.js:48:31
    at process._tickCallback (internal/process/next_tick.js:68:7)
  name: 'FetchError',
  message:
   'invalid json response body at http://react.zpao.com/builds/master/_commits/d827445a500e497712a1a404c7ccc25779b1df5c/results.json reason: Unexpected token < in JSON at position 0',
  type: 'invalid-json' }

Generated by 🚫 dangerJS

@edmorley

edmorley commented Sep 8, 2019

Copy link
Copy Markdown

Thank you for opening a PR for this! I'm presuming the CI failure is an infra issue and unrelated to this change? If so, could someone retrigger CI? :-)

@SimenB

SimenB commented Sep 8, 2019

Copy link
Copy Markdown
Contributor Author

Rebased this and added the second rule

@sizebot

sizebot commented Sep 8, 2019

Copy link
Copy Markdown

No significant bundle size changes to report.

Generated by 🚫 dangerJS against 65b466d

@stale

stale Bot commented Jan 9, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution.

@stale stale Bot added the Resolution: Stale Automatically closed due to inactivity label Jan 9, 2020
@necolas necolas requested a review from gaearon January 9, 2020 21:57
@stale stale Bot removed the Resolution: Stale Automatically closed due to inactivity label Jan 9, 2020
@gaearon gaearon merged commit 79a2512 into react:master Feb 28, 2020
@gaearon

gaearon commented Feb 28, 2020

Copy link
Copy Markdown
Collaborator

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants