chore: update code owners for easier release management (#1894)

* chore: update code owners for easier release management

* chore: add slash to directories to apply the changes to all

* chore: pass env correctly to npm

* chore: use cross-env

* chore: try direct commit hash

* chore: specify head commit too

* chore: use env now

* chore: move base ref to global env

* chore: check for affected projects

* chore: fix all the workflows

* chore: skip fixing workflows for now

* chore: fix affected issues

.github/workflows/security-code-quality-and-bundle-size-checks.yml

@@ -7,6 +7,7 @@ on:
 
 env:
   NODE_OPTIONS: "--no-warnings"
+  BASE_REF: ${{ github.event.pull_request.base.sha || 'HEAD' }}
 
 jobs:
   bundle-size-checks:
@@ -64,15 +65,11 @@ jobs:
           npm run check:duplicates
 
       - name: Build the project
-        env:
-          BASE_REF: ${{ github.event.pull_request.base.sha }}
         run: |
           npm run build:ci
           npm run build:modern:ci
 
       - name: Execute package linting checks
-        env:
-          BASE_REF: ${{ github.event.pull_request.base.sha }}
         run: |
           npm run check:pub:ci
 

.github/workflows/unit-tests-and-lint.yml

@@ -10,6 +10,7 @@ on:
 
 env:
   NODE_OPTIONS: "--no-warnings"
+  BASE_REF: ${{ github.event.pull_request.base.sha || 'HEAD' }}
 
 jobs:
   test:
@@ -35,9 +36,14 @@ jobs:
         run: |
           npm run setup:ci
 
+      - name: Check for affected projects
+        id: check_affected
+        run: |
+          AFFECTED_PROJECTS=$(npx nx show projects --affected --base=$BASE_REF | tr '\n' ',')
+          echo "Affected projects: $AFFECTED_PROJECTS"
+          echo "affected_projects=$AFFECTED_PROJECTS" >> $GITHUB_OUTPUT
+
       - name: Execute unit tests
-        env:
-          BASE_REF: ${{ github.event.pull_request.base.sha || 'HEAD' }}
         run: |
           npm run test:ci
 
@@ -49,8 +55,6 @@ jobs:
           files: ./packages/analytics-js/reports/coverage/clover.xml,./packages/analytics-js-common/reports/coverage/clover.xml,./packages/analytics-js-integrations/reports/coverage/clover.xml,./packages/analytics-js-plugins/reports/coverage/clover.xml,./packages/analytics-js-service-worker/reports/coverage/clover.xml,./packages/analytics-v1.1/reports/coverage/clover.xml,./packages/analytics-js-cookies/reports/coverage/clover.xml
 
       - name: Execute linting check
-        env:
-          BASE_REF: ${{ github.event.pull_request.base.sha || 'HEAD' }}
         run: |
           npm run check:lint:ci
 

CODEOWNERS

@@ -1,3 +1,12 @@
 * @rudderlabs/js-sdk
-/packages/analytics-js-integrations/ @rudderlabs/js-sdk-dm-intg-eng
-/packages/analytics-js-common/src/constants/integrations @rudderlabs/js-sdk-dm-intg-eng
+
+/packages/analytics-js-integrations/src/ @rudderlabs/js-sdk-dm-intg-eng
+/packages/analytics-js-integrations/__fixtures__/ @rudderlabs/js-sdk-dm-intg-eng
+/packages/analytics-js-integrations/__tests__/ @rudderlabs/js-sdk-dm-intg-eng
+/packages/analytics-js-integrations/__mocks__/ @rudderlabs/js-sdk-dm-intg-eng
+/packages/analytics-js-integrations/README.md @rudderlabs/js-sdk-dm-intg-eng
+/packages/analytics-js-integrations/.size-limit.js @rudderlabs/js-sdk-dm-intg-eng
+
+/packages/analytics-js-common/src/constants/integrations/ @rudderlabs/js-sdk-dm-intg-eng
+/assets/integrations/AdobeAnalytics/ @rudderlabs/js-sdk-dm-intg-eng
+/examples/integrations/ @rudderlabs/js-sdk-dm-intg-eng

package.json

@@ -144,6 +144,7 @@
     "commitizen": "4.3.1",
     "commitlint": "19.5.0",
     "core-js": "3.38.1",
+    "cross-env": "7.0.3",
     "dotenv": "16.4.5",
     "each": "2.7.0",
     "eslint": "8.57.0",