-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yangshun has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! You should investigate why Travis CI is failing though...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yangshun has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: **Summary** Now `nvm install 8` ends up installing npm 6.4.1 but our `package.json` doesn't support that npm version. This PR includes 6.x in the versions we support. More context: #1864 (comment) **Test Plan** CI tests. Pull Request resolved: #1866 Reviewed By: niveditc Differential Revision: D9789746 Pulled By: niveditc fbshipit-source-id: 8bdf8573c84d66bdce8281c4b2660df953399583
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yangshun has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: **Summary** Now `nvm install 8` ends up installing npm 6.4.1 but our `package.json` doesn't support that npm version. This PR includes 6.x in the versions we support. More context: facebookarchive#1864 (comment) **Test Plan** CI tests. Pull Request resolved: facebookarchive#1866 Reviewed By: niveditc Differential Revision: D9789746 Pulled By: niveditc fbshipit-source-id: 8bdf8573c84d66bdce8281c4b2660df953399583
Summary: **Summary** Add Algolia search as per facebookarchive#1775. **Test Plan** <img width="1552" alt="screen shot 2018-09-12 at 10 07 59 am" src="https://user-images.githubusercontent.com/1315101/45397732-fa33e780-b673-11e8-8264-5044e82fbe61.png"> Clicking on the results brings me to the correct location in the docs. It's magic! Pull Request resolved: facebookarchive#1864 Reviewed By: niveditc Differential Revision: D9784511 Pulled By: niveditc fbshipit-source-id: 4e1b0b35af85ab349d79a83315f2010bc32c7f7f
Summary: **Summary** Add Algolia search as per #1775. **Test Plan** <img width="1552" alt="screen shot 2018-09-12 at 10 07 59 am" src="https://user-images.githubusercontent.com/1315101/45397732-fa33e780-b673-11e8-8264-5044e82fbe61.png"> Clicking on the results brings me to the correct location in the docs. It's magic! Pull Request resolved: facebookarchive/draft-js#1864 Reviewed By: niveditc Differential Revision: D9784511 Pulled By: niveditc fbshipit-source-id: 4e1b0b35af85ab349d79a83315f2010bc32c7f7f
Summary: **Summary** Add Algolia search as per #1775. **Test Plan** <img width="1552" alt="screen shot 2018-09-12 at 10 07 59 am" src="https://user-images.githubusercontent.com/1315101/45397732-fa33e780-b673-11e8-8264-5044e82fbe61.png"> Clicking on the results brings me to the correct location in the docs. It's magic! Pull Request resolved: facebookarchive/draft-js#1864 Reviewed By: niveditc Differential Revision: D9784511 Pulled By: niveditc fbshipit-source-id: 4e1b0b35af85ab349d79a83315f2010bc32c7f7f
Summary
Add Algolia search as per #1775.
Fixes #1775.
Test Plan
Clicking on the results brings me to the correct location in the docs. It's magic!