Merge branch 'develop' into abhi-remove-sentry-tracing

Author: AbhiPrasad

.craft.yml

@@ -131,12 +131,7 @@ targets:
     id: '@sentry-internal/eslint-config-sdk'
     includeNames: /^sentry-internal-eslint-config-sdk-\d.*\.tgz$/
 
-  ## 8. Deprecated packages we still release (but no packages depend on them anymore)
-  - name: npm
-    id: '@sentry/hub'
-    includeNames: /^sentry-hub-\d.*\.tgz$/
-
-  ## 9. Experimental packages
+  ## 8. Experimental packages
   - name: npm
     id: '@sentry/node-experimental'
     includeNames: /^sentry-node-experimental-\d.*\.tgz$/

.github/workflows/build.yml

@@ -715,6 +715,13 @@ jobs:
           PW_BUNDLE: ${{ matrix.bundle }}
         working-directory: dev-packages/browser-integration-tests
         run: yarn test:ci${{ matrix.project && format(' --project={0}', matrix.project) || '' }}${{ matrix.shard && format(' --shard={0}/{1}', matrix.shard, matrix.shards) || '' }}
+      - name: Upload Playwright Traces
+        uses: actions/upload-artifact@v3
+        if: always()
+        with:
+          name: playwright-traces
+          path: dev-packages/browser-integration-tests/test-results
+
 
   job_browser_loader_tests:
     name: Playwright Loader (${{ matrix.bundle }}) Tests
@@ -772,6 +779,12 @@ jobs:
         run: |
           cd dev-packages/browser-integration-tests
           yarn test:loader
+      - name: Upload Playwright Traces
+        uses: actions/upload-artifact@v3
+        if: always()
+        with:
+          name: playwright-traces
+          path: dev-packages/browser-integration-tests/test-results
 
   job_browser_integration_tests:
     name: Browser (${{ matrix.browser }}) Tests

dev-packages/browser-integration-tests/playwright.config.ts

@@ -10,6 +10,10 @@ const config: PlaywrightTestConfig = {
   workers: process.env.CI ? 3 : undefined,
   testMatch: /test.ts/,
 
+  use: {
+    trace: process.env.CI ? 'retry-with-trace' : 'off',
+  },
+
   projects: [
     {
       name: 'chromium',

dev-packages/e2e-tests/verdaccio-config/config.yaml

@@ -194,12 +194,6 @@ packages:
     unpublish: $all
     # proxy: npmjs # Don't proxy for E2E tests!
 
-  '@sentry/hub':
-    access: $all
-    publish: $all
-    unpublish: $all
-    # proxy: npmjs # Don't proxy for E2E tests!
-
   '@sentry-internal/*':
     access: $all
     publish: $all

package.json

@@ -57,7 +57,6 @@
     "packages/eslint-plugin-sdk",
     "packages/feedback",
     "packages/gatsby",
-    "packages/hub",
     "packages/integrations",
     "packages/integration-shims",
     "packages/nextjs",