Skip to content

feat: Add Sentry to the bundler plugins #224

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 23 commits into from
Jan 8, 2025

bring back early returns if no provider or upload url

e5ba3cc
Select commit
Loading
Failed to load commit list.
Merged

feat: Add Sentry to the bundler plugins #224

bring back early returns if no provider or upload url
e5ba3cc
Select commit
Loading
Failed to load commit list.
Codecov Notifications / codecov/patch failed Jan 7, 2025 in 1s

58.69% of diff hit (target 83.54%)

View this Pull Request on Codecov

58.69% of diff hit (target 83.54%)

Annotations

Check warning on line 46 in packages/astro-plugin/src/astro-bundle-analysis/astroBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/astro-plugin/src/astro-bundle-analysis/astroBundleAnalysisPlugin.ts#L46 

Added line #L46 was not covered by tests

Check warning on line 20 in packages/astro-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/astro-plugin/src/index.ts#L20 

Added line #L20 was not covered by tests

Check warning on line 54 in packages/astro-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/astro-plugin/src/index.ts#L45-L54 

Added lines #L45 - L54 were not covered by tests

Check warning on line 72 in packages/astro-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/astro-plugin/src/index.ts#L57-L72 

Added lines #L57 - L72 were not covered by tests

Check warning on line 71 in packages/bundler-plugin-core/src/sentry/telemetry.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/telemetry.ts#L71 

Added line #L71 was not covered by tests

Check warning on line 74 in packages/bundler-plugin-core/src/sentry/telemetry.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/telemetry.ts#L73-L74 

Added lines #L73 - L74 were not covered by tests

Check warning on line 78 in packages/bundler-plugin-core/src/sentry/telemetry.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/telemetry.ts#L76-L78 

Added lines #L76 - L78 were not covered by tests

Check warning on line 83 in packages/bundler-plugin-core/src/sentry/telemetry.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/telemetry.ts#L81-L83 

Added lines #L81 - L83 were not covered by tests

Check warning on line 101 in packages/bundler-plugin-core/src/sentry/telemetry.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/telemetry.ts#L94-L101 

Added lines #L94 - L101 were not covered by tests

Check warning on line 53 in packages/bundler-plugin-core/src/sentry/transports.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/transports.ts#L51-L53 

Added lines #L51 - L53 were not covered by tests

Check warning on line 86 in packages/bundler-plugin-core/src/sentry/transports.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/sentry/transports.ts#L86 

Added line #L86 was not covered by tests

Check warning on line 203 in packages/bundler-plugin-core/src/utils/Output.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/bundler-plugin-core/src/utils/Output.ts#L202-L203 

Added lines #L202 - L203 were not covered by tests

Check warning on line 21 in packages/nextjs-webpack-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/index.ts#L21 

Added line #L21 was not covered by tests

Check warning on line 23 in packages/nextjs-webpack-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/index.ts#L23 

Added line #L23 was not covered by tests

Check warning on line 58 in packages/nextjs-webpack-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/index.ts#L49-L58 

Added lines #L49 - L58 were not covered by tests

Check warning on line 65 in packages/nextjs-webpack-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/index.ts#L61-L65 

Added lines #L61 - L65 were not covered by tests

Check warning on line 72 in packages/nextjs-webpack-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/index.ts#L71-L72 

Added lines #L71 - L72 were not covered by tests

Check warning on line 18 in packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts#L18 

Added line #L18 was not covered by tests

Check warning on line 28 in packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts#L27-L28 

Added lines #L27 - L28 were not covered by tests

Check warning on line 38 in packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts#L37-L38 

Added lines #L37 - L38 were not covered by tests

Check warning on line 46 in packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts#L41-L46 

Added lines #L41 - L46 were not covered by tests

Check warning on line 75 in packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts#L75 

Added line #L75 was not covered by tests

Check warning on line 106 in packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nextjs-webpack-plugin/src/nextjs-webpack-bundle-analysis/nextJSWebpackBundleAnalysisPlugin.ts#L106 

Added line #L106 was not covered by tests

Check warning on line 21 in packages/nuxt-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nuxt-plugin/src/index.ts#L21 

Added line #L21 was not covered by tests

Check warning on line 50 in packages/nuxt-plugin/src/index.ts

See this annotation in the file changed.

@codecov-notifications codecov-notifications / codecov/patch

packages/nuxt-plugin/src/index.ts#L41-L50 

Added lines #L41 - L50 were not covered by tests
View more details on Codecov Notifications