search

nytimes / react-tracking

🎯 Declarative tracking for React apps.
https://open.nytimes.com/introducing-react-tracking-declarative-tracking-for-react-apps-2c76706bb79a
Other
1.88k stars 123 forks source link

Add additional peer dependency to allow use with react 18 #203

Closed AnthonyCrowcroft closed 2 years ago

AnthonyCrowcroft commented 2 years ago

Hey really keen to keep using this library but we are shifting up to react 18 so am needing to get this added in, all the tests are still passing as expected and the devDependency remains unchanged as no newer functionality is required therefore no older peer dependencies are required to be removed.

TLDR; very minor change that lets more people use the library :)

AnthonyCrowcroft commented 2 years ago

@tizmagik you mentioned this being something you were happy to look at a PR for but I am new to contributing to this project any additional things I need to do to get this merged in for me to use. 🚀

tizmagik commented 2 years ago

Oh I'm sorry I missed this earlier. Let me merge and cut a new release later today!

AnthonyCrowcroft commented 2 years ago

@tizmagik if at all possible please post a comment here so I can pull the new version into my project.

tizmagik commented 2 years ago

Published as v9.2.1 -- thanks again @AnthonyCrowcroft !