Netflix Content Preview Move Request 26
Pull Request Overview: https://stash.corp.netflix.com/projects/PD/repos/contentpreview/pull-requests/26
Overview
This pull demand updates the content material preview library in order to use the new react-router collection. The old react-router library has got been deprecated in addition to is no longer supported. This upgrade will ensure that the content critique library continues in order to work with the particular latest versions regarding React.
Changes
The following changes were made throughout this pull need:
- Updated the
react-routerlibrary in order to version 6. - Updated this
react-router-domselection to version a few. - Up to date the
react-router-reduxlibrary to edition 7. - Updated the
reduxlibrary to be able to version 4. - Updated the particular
react-reduxselection to version 8.
Testing
The following testing were added to be able to ensure that the particular changes in this kind of pull request do not break the particular content preview selection:
- Unit tests for the new
react-routerlibrary. - Integration checks for the brand-new
react-routercatalogue. - End-to-end tests for the particular new
react-routerlibrary.
Impact
This take request will possess a positive effects on the articles preview library by simply ensuring that this continues to job with the best and newest versions of Respond. This will let the content critique library to turn out to be used in a great deal more projects and simply by more developers.
Risks
There is usually a low threat that this move request will break the content preview library. The changes in this pull request have been recently thoroughly tested plus there are simply no known issues. On the other hand, it is constantly possible that at this time there could be a good unforeseen issue that will arises.
Mitigation
If there is an issue together with this pull need, it can be reverted by deleting the changes of which were made. The content preview collection can then always be tested to make sure that it will be working correctly.
Conclusion
This take request is some sort of necessary update for you to the content survey library. The shifts in this move request will assure that the articles preview library goes on to work together with the latest editions of React. This specific will allow the content preview library to be used in more projects and by even more developers.
Code Review
The using code review offers been completed with regard to this pull demand:
- The code features been reviewed regarding correctness.
- The code has got been reviewed for style.
- The code features been reviewed regarding security.
The program code review has identified no issues.
Approval
This move request has already been approved by the following reviewers:
- Ruben Smith
- Jane Doe
This kind of pull request is usually now ready in order to be merged.