Add missing dependency added in #132

[T0RAT0RA]

Sorry I forgot to add react-router-dom in #132

✔️ Checklist

• Added tests for new functionality and regression tests for bug fixes
• Added changeset (run yarn changeset in the root)
• Screenshots of before and after attached (for UI changes)
• Added or updated documentation (if applicable)

[jvilimek]

No worries. The lint step should have failed in the previous PR during CI phase.. but maybe the sources were not synchronized, this is something I have added just recently (#126)

[T0RAT0RA]

It's still failing but I don't know why. I tried a bunch of different updates on the yarn.lock and package.json files, but I don't know what's going on.

[jvilimek]

I believe the "react-router-dom": "6.4.3" needs to be added to devDependencies to the plugin as it is now used in tests

UPDATE:

you may check by running locally yarn backstage-cli repo lint:

This will fix it:

[jvilimek]

I have updated my comment above. Shall I fix it? I can have PR up and running in no time. It shall be one-liner in one file.

[T0RAT0RA]

Shall I fix it?

Yes please, thanks!
there is something wrong with my setup, the linter doesn't pick up the issue. I'l have to investigate later.

[jvilimek]

sure, np. Created #145 . Closing this one.