Migration guide feedback

[wojtekmaj]

• It was unclear for me whether I should upgrade react-router and/or react-router-dom when installing react-router-dom-v5-compat. It only got clear when I read through the entire document.
• Is there a good reason why we should use useLocation from react-router-dom-v5-compat? Should I "upgrade" pre-existing useLocation uses from RRv5 to be imported from react-router-dom-v5-compat instead?
• It should be mentioned how to switch fromhistory.push({ pathname: '/foo', state: { some: 'state' }).
• In some cases, when <Switch> was used in a nested component, using CompatRoute may not be enough for some paths to be rendered fully on the screen. It may only be fixed in Step 7!
• In "Convert Switch to Routes" step, invalid diff is shown: it points to remove Route imports, where CompatRoute is the one actually being used.
• I feel like "Redirect when no other routes are matching" is an use case popular enough to be included in the docs. In fact, there's no mention of Redirect at all!
• Copying & pasting git grep -lz src | xargs -0 sed -i '' -e 's/react-router-dom-v5-compat/react-router-dom/g' resulted in "sed: RE error: illegal byte sequence" error for me