Merge pull request #225 from splitio/react_namespace_import

Updated `React` default imports to the recommended namespace imports

Author: EmilianoSanchez

.github/workflows/ci-cd.yml

@@ -78,7 +78,7 @@ jobs:
 
       - name: Store assets
         if: github.event_name == 'push' && (github.ref == 'refs/heads/master' || github.ref == 'refs/heads/development')
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: assets
           path: umd/
@@ -100,7 +100,7 @@ jobs:
 
     steps:
       - name: Download assets
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           name: assets
           path: umd
@@ -110,7 +110,7 @@ jobs:
         working-directory: umd
 
       - name: Configure AWS credentials
-        uses: aws-actions/configure-aws-credentials@v1-node16
+        uses: aws-actions/configure-aws-credentials@v4
         with:
           role-to-assume: arn:aws:iam::${{ matrix.account_id }}:role/gha-public-assets-role
           aws-region: us-east-1
@@ -139,7 +139,7 @@ jobs:
 
     steps:
       - name: Download assets
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           name: assets
           path: umd
@@ -149,7 +149,7 @@ jobs:
         working-directory: umd
 
       - name: Configure AWS credentials
-        uses: aws-actions/configure-aws-credentials@v1-node16
+        uses: aws-actions/configure-aws-credentials@v4
         with:
           role-to-assume: arn:aws:iam::${{ matrix.account_id }}:role/gha-public-assets-role
           aws-region: us-east-1

CHANGES.txt

@@ -1,3 +1,6 @@
+2.0.2 (January XX, 2025)
+ - Updated the internal imports of React library from default to namespace imports as this is the recommended approach for better compatibility with React, TypeScript, ES modules, and tree shaking (https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#removing-unused-react-imports).
+
 2.0.1 (December 4, 2024)
  - Updated @splitsoftware/splitio package to version 11.0.3 that includes some improvements and bugfixes.
  - Updated internal handling of the `updateOnSdkTimedout` param to remove the wrong log "[ERROR] A listener was added for SDK_READY_TIMED_OUT on the SDK, which has already fired and won't be emitted again".

LICENSE

@@ -1,4 +1,4 @@
-Copyright © 2024 Split Software, Inc.
+Copyright © 2025 Split Software, Inc.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio-react",
-  "version": "2.0.1",
+  "version": "2.0.2-rc.0",
   "description": "A React library to easily integrate and use Split JS SDK",
   "main": "cjs/index.js",
   "module": "esm/index.js",

package.json

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { SplitContext } from './SplitContext';
 import { ISplitClientProps } from './types';
 import { useSplitClient } from './useSplitClient';

src/SplitClient.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { ISplitContextValues } from './types';
 import { EXCEPTION_NO_SFP } from './constants';
 

src/SplitContext.ts

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 
 import { ISplitFactoryProviderProps } from './types';
 import { VERSION, WARN_SF_CONFIG_AND_FACTORY } from './constants';

src/SplitFactoryProvider.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 
 import { SplitContext } from './SplitContext';
 import { ISplitTreatmentsProps } from './types';

src/SplitTreatments.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render, act } from '@testing-library/react';
 
 /** Mocks and test utils */

src/__tests__/SplitClient.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render } from '@testing-library/react';
 import { SplitContext } from '../SplitContext';
 import { SplitFactoryProvider } from '../SplitFactoryProvider';

src/__tests__/SplitContext.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render, act } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/SplitFactoryProvider.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render, RenderResult, act } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/SplitTreatments.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render } from '@testing-library/react';
 import { ISplitStatus } from '../../types';
 const { SplitFactory: originalSplitFactory } = jest.requireActual('@splitsoftware/splitio/client');

src/__tests__/testUtils/utils.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { act, render } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/useSplitClient.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { act, render } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/useSplitManager.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { act, render } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/useSplitTreatments.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/withSplitClient.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { render } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/withSplitFactory.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { act, render } from '@testing-library/react';
 
 /** Mocks */

src/__tests__/withSplitTreatments.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { useSplitContext } from './SplitContext';
 import { getSplitClient, initAttributes, getStatus } from './utils';
 import { ISplitContextValues, IUseSplitClientOptions } from './types';

src/useSplitClient.ts

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { memoizeGetTreatmentsWithConfig } from './utils';
 import { ISplitTreatmentsChildProps, IUseSplitTreatmentsOptions } from './types';
 import { useSplitClient } from './useSplitClient';

src/useSplitTreatments.ts

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { ISplitClientChildProps } from './types';
 import { SplitClient } from './SplitClient';
 

src/withSplitClient.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { ISplitFactoryChildProps } from './types';
 import { SplitFactoryProvider } from './SplitFactoryProvider';
 import { SplitClient } from './SplitClient';

src/withSplitFactory.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import * as React from 'react';
 import { ISplitTreatmentsChildProps } from './types';
 import { SplitTreatments } from './SplitTreatments';