removed 'commit_hash' env var, that is not used anymore

Author: EmilianoSanchez

.github/workflows/ci.yml

@@ -43,7 +43,7 @@ jobs:
         run: npm run test -- --coverage
 
       - name: npm build
-        run: BUILD_BRANCH=$(echo "${GITHUB_REF#refs/heads/}") BUILD_COMMIT=${{ github.sha }} npm run build
+        run: BUILD_BRANCH=$(echo "${GITHUB_REF#refs/heads/}") npm run build
 
       - name: Set VERSION env
         run: echo "VERSION=$(cat package.json | jq -r .version)" >> $GITHUB_ENV

CHANGES.txt

@@ -1,3 +1,6 @@
+1.9.1 (September XX, 2023)
+ - Updated linter and other dependencies for vulnerability fixes.
+
 1.9.0 (July 18, 2023)
  - Updated some transitive dependencies for vulnerability fixes.
  - Updated @splitsoftware/splitio package to version 10.23.0 that includes:

package.json

@@ -19,7 +19,7 @@
   "scripts": {
     "build:cjs": "rimraf lib/* types/* && tsc -m commonjs --outDir lib -d true --declarationDir types",
     "build:esm": "rimraf es/* && tsc",
-    "build:umd": "rimraf umd/* && webpack --config webpack.dev.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && webpack --config webpack.prod.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT",
+    "build:umd": "rimraf umd/* && webpack --config webpack.dev.js --env branch=$BUILD_BRANCH && webpack --config webpack.prod.js --env branch=$BUILD_BRANCH",
     "build": "npm run build:cjs && npm run build:esm && npm run build:umd",
     "postbuild": "replace 'REACT_SDK_VERSION_NUMBER' $npm_package_version ./lib/constants.js ./es/constants.js ./umd -r",
     "check": "npm run check:lint && npm run check:types",

src/SplitClient.tsx

@@ -22,7 +22,7 @@ export class SplitComponent extends React.Component<IUpdateProps & { factory: Sp
 
   // Using `getDerivedStateFromProps` since the state depends on the status of the client in props, which might change over time.
   // It could be avoided by removing the client and its status from the component state.
-  // But it implies to have another instance property to use instead of the state, because we need a unique reference value for SplitContext.Producer
+  // But it implies to have another instance property to use instead of the state, because we need a unique reference value for SplitContext.Provider
   static getDerivedStateFromProps(props: ISplitClientProps & { factory: SplitIO.IBrowserSDK | null, client: SplitIO.IBrowserClient | null }, state: ISplitContextValues) {
     const { client, factory, attributes } = props;
     // initAttributes can be called in the `render` method too, but it is better here for separation of concerns