update ci-cd to simplify CDN RC deployment

Author: EmilianoSanchez

.github/workflows/ci-cd.yml

@@ -44,7 +44,7 @@ jobs:
         run: npm run test
 
       - 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: Store assets
         if: ${{ github.event_name == 'push' && (github.ref == 'refs/heads/development' || github.ref == 'refs/heads/main') }}

.github/workflows/sonar-scan.yml

@@ -32,7 +32,7 @@ jobs:
         run: npm run check
 
       - 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: SonarQube Scan (Push)
         if: github.event_name == 'push'

package-lock.json

@@ -26,7 +26,7 @@
     "build:esm": "rimraf esm && tsc -outDir esm && ./scripts/build_esm_replace_imports.sh",
     "build:cjs": "rimraf cjs && tsc -outDir cjs -m CommonJS && ./scripts/build_cjs_replace_imports.sh",
     "build:umd-visualizer": "rollup --config rollup.visualizer.config.js",
-    "build:umd": "rollup --config rollup.ci.config.js --branch=$BUILD_BRANCH --commit_hash=$BUILD_COMMIT",
+    "build:umd": "rollup --config rollup.ci.config.js --branch=$BUILD_BRANCH",
     "build:ga-to-split-autorequire": "terser ./node_modules/@splitsoftware/splitio-commons/src/integrations/ga/autoRequire.js --mangle --output ./scripts/ga-to-split-autorequire.js",
     "test": "npm run test:unit && npm run test:e2e",
     "test:unit": "jest",

package.json

@@ -3,7 +3,7 @@ import terser from '@rollup/plugin-terser';
 
 export default env => {
 
-  const fileName = (outputSuffix) => `split-browser-${env.branch !== 'main' ? `dev-${env.commit_hash || VERSION}` : VERSION}${outputSuffix ? `.${outputSuffix}` : ''}`;
+  const fileName = (outputSuffix) => `split-browser-${env.branch !== 'main' ? `dev-${VERSION}` : VERSION}${outputSuffix ? `.${outputSuffix}` : ''}`;
 
   const createRollupConfig = (input, outputSuffix) => ({
     input,

rollup.ci.config.js

@@ -137,7 +137,7 @@ interface ILoggerAPI {
  */
 interface IUserConsentAPI {
   /**
-   * Set or update the user consent status. Possible values are `true` and `false`, which represent user consent `'GRANTED'` and `'DECLINED'` respectively.
+   * Sets or updates the user consent status. Possible values are `true` and `false`, which represent user consent `'GRANTED'` and `'DECLINED'` respectively.
    * - `true ('GRANTED')`: the user has granted consent for tracking events and impressions. The SDK will send them to Split cloud.
    * - `false ('DECLINED')`: the user has declined consent for tracking events and impressions. The SDK will not send them to Split cloud.
    *
@@ -149,7 +149,7 @@ interface IUserConsentAPI {
    */
   setStatus(userConsent: boolean): boolean;
   /**
-   * Get the user consent status.
+   * Gets the user consent status.
    *
    * @function getStatus
    * @returns {ConsentStatus} The user consent status.