Support React v18

[outbackStack]

Do you plan to upgrade to React 18 to avoid npm dependency conflict?

[andipaetzold]

I just created a PR to add React 18 as a peer dependency: #164
I don't see any change in the new release that would break react-helmet-async.

[igorlino]

I dont' have a dependency conflict, but a 2 types of TypeScript errors in my CRA-based application

S2769: No overload matches this call.
  Overload 1 of 2, '(props: ProviderProps | Readonly<ProviderProps>): HelmetProvider', gave the following error.
    Type '{ children: Element[]; }' has no properties in common with type 'IntrinsicAttributes & IntrinsicClassAttributes<HelmetProvider> & Readonly<ProviderProps>'.
  Overload 2 of 2, '(props: ProviderProps, context: any): HelmetProvider', gave the following error.
    Type '{ children: Element[]; }' has no properties in common with type 'IntrinsicAttributes & IntrinsicClassAttributes<HelmetProvider> & Readonly<ProviderProps>'.
    21 | function App() {
    22 |     return (
  > 23 |         <HelmetProvider>

TS2769: No overload matches this call.
  Overload 1 of 2, '(props: HelmetProps | Readonly<HelmetProps>): Helmet', gave the following error.
    Type '{ children: Element; }' has no properties in common with type 'IntrinsicAttributes & IntrinsicClassAttributes<Helmet> & Readonly<HelmetProps>'.
  Overload 2 of 2, '(props: HelmetProps, context: any): Helmet', gave the following error.
    Type '{ children: Element; }' has no properties in common with type 'IntrinsicAttributes & IntrinsicClassAttributes<Helmet> & Readonly<HelmetProps>'.
    22 |     return (
    23 |         <HelmetProvider>
  > 24 |             <Helmet>

[igorlino]

So, in summary, react-helmet-sync should be migrated also because fo this:

https://reactjs.org/blog/2022/03/08/react-18-upgrade-guide.html#updates-to-typescript-definitions

This is definitively a blocker for whoever is using TypeScript-based react projects.

[andipaetzold]

@igorlino That's also covered in #164.
If the peer dependency conflict is not an issue for you, I'd recommend using patch-package until the PR is merged and a new version is released.

[suren-atoyan]

@staylor could you please check the PR? ☝️