Skip to content

fix(ci): Update validate-pr action to remove draft enforcement - #20037

Merged
stephanie-anderson merged 3 commits into
developfrom
fix/update-validate-pr-action
Mar 30, 2026
Merged

fix(ci): Update validate-pr action to remove draft enforcement#20037
stephanie-anderson merged 3 commits into
developfrom
fix/update-validate-pr-action

Conversation

@stephanie-anderson

@stephanie-andersonstephanie-anderson commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

The validate-pr action's draft enforcement step was failing with:

API call failed: GraphQL: Resource not accessible by integration (convertPullRequestToDraft)

Draft enforcement has been removed from the shared action in getsentry/github-workflows#159. This bumps the pinned SHA.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

stephanie-andersonand others added 2 commits March 30, 2026 10:32
The draft enforcement step was failing with `Resource not accessible by
integration (convertPullRequestToDraft)` due to insufficient app
permissions. The draft conversion has been removed from the shared
action.
Refs getsentry/github-workflows#159
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@stephanie-andersonstephanie-anderson changed the title Fix/update validate pr actionfix(ci): Update validate-pr action to remove draft enforcementMar 30, 2026
@github-actions

github-actionsBot commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Deps

  • Bump babel-loader from 10.0.0 to 10.1.1 by dependabot in #19997
  • Bump handlebars from 4.7.7 to 4.7.9 by dependabot in #20008

Nuxt

  • Add middleware instrumentation compatibility for Nuxt 5 by s1gr1d in #19968
  • Support parametrized SSR routes in Nuxt 5 by s1gr1d in #19977

Other

  • (browser) Replace element timing spans with metrics by logaretm in #19869
  • (bun) Add bunRuntimeMetricsIntegration by chargome in #19979
  • (core) Support embedding APIs in google-genai by nicohrubec in #19797
  • (node) Add nodeRuntimeMetricsIntegration by chargome in #19923
  • (node-core) Add OTLP integration for node-core/light by andreiborza in #19729
  • (solid) Add route parametrization for Solid Router by andreiborza in #20031

Bug Fixes 🐛

Ci

  • Update validate-pr action to remove draft enforcement by stephanie-anderson in #20037
  • Update validate-pr action to remove draft enforcement by stephanie-anderson in #20035

Other

  • (e2e) Pin @opentelemetry/api to 1.9.0 in ts3.8 test app by logaretm in #19992
  • (node) Ensure startNewTrace propagates traceId in OTel environments by logaretm in #19963
  • (nuxt) Use virtual module for Nuxt pages data (SSR route parametrization) by s1gr1d in #20020
  • (opentelemetry) Convert seconds timestamps in span.end() to milliseconds by logaretm in #19958

Documentation 📚

  • (release) Update publishing-a-release.md by nicohrubec in #19982

Internal Changes 🔧

Core

  • Introduce instrumented method registry for AI integrations by nicohrubec in #19981
  • Consolidate getOperationName into one shared utility by nicohrubec in #19971

Deps

  • Bump amqplib from 0.10.7 to 0.10.9 by dependabot in #20000
  • Bump actions/upload-artifact from 6 to 7 by dependabot in #19569
  • Bump srvx from 0.11.12 to 0.11.13 by dependabot in #20001
  • Bump @apollo/server from 5.4.0 to 5.5.0 by dependabot in #20007

Deps Dev

  • Remove esbuild override in astro-5-cf-workers E2E test by isaacs in #20024
  • Bump node-forge from 1.3.2 to 1.4.0 by dependabot in #20012
  • Bump yaml from 2.8.2 to 2.8.3 by dependabot in #19985

Other

  • (deno) Expand Deno E2E test coverage by chargome in #19957
  • (e2e) Add e2e tests for nodeRuntimeMetricsIntegration by chargome in #19989

🤖 This preview updates automatically when you update the PR.

@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

PathSize% ChangeChange
@sentry/browser25.69 kB+0.04%+8 B 🔺
@sentry/browser - with treeshaking flags24.18 kB+0.04%+8 B 🔺
@sentry/browser (incl. Tracing)42.19 kB+0.04%+16 B 🔺
@sentry/browser (incl. Tracing, Profiling)46.8 kB+0.02%+6 B 🔺
@sentry/browser (incl. Tracing, Replay)80.99 kB+0.02%+12 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.6 kB+0.01%+7 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas)85.72 kB+0.02%+15 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback)97.98 kB+0.01%+9 B 🔺
@sentry/browser (incl. Feedback)42.48 kB+0.02%+8 B 🔺
@sentry/browser (incl. sendFeedback)30.36 kB+0.04%+10 B 🔺
@sentry/browser (incl. FeedbackAsync)35.41 kB+0.03%+8 B 🔺
@sentry/browser (incl. Metrics)26.98 kB+0.1%+26 B 🔺
@sentry/browser (incl. Logs)27.13 kB+0.12%+31 B 🔺
@sentry/browser (incl. Metrics & Logs)27.81 kB+0.11%+30 B 🔺
@sentry/react27.46 kB+0.05%+11 B 🔺
@sentry/react (incl. Tracing)44.53 kB+0.03%+10 B 🔺
@sentry/vue30.13 kB+0.02%+5 B 🔺
@sentry/vue (incl. Tracing)44.09 kB+0.03%+9 B 🔺
@sentry/svelte25.71 kB+0.04%+9 B 🔺
CDN Bundle28.39 kB--
CDN Bundle (incl. Tracing)43.2 kB--
CDN Bundle (incl. Logs, Metrics)29.76 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.25 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.56 kB--
CDN Bundle (incl. Tracing, Replay)80.08 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)81.16 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)85.62 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.67 kB--
CDN Bundle - uncompressed82.93 kB--
CDN Bundle (incl. Tracing) - uncompressed128.07 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed87.07 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed131.48 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed210.06 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed244.95 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed248.34 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed257.86 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed261.25 kB--
@sentry/nextjs (client)46.94 kB+0.03%+11 B 🔺
@sentry/sveltekit (client)42.68 kB+0.03%+12 B 🔺
@sentry/node-core56.52 kB+0.04%+19 B 🔺
@sentry/node173.62 kB+0.02%+22 B 🔺
@sentry/node - without tracing96.54 kB+0.01%+7 B 🔺
@sentry/aws-serverless113.54 kB+0.01%+6 B 🔺

View base workflow run

@github-actions

Copy link
Copy Markdown
Contributor

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,556-9,392-9%
GET With Sentry1,51618%1,746-13%
GET With Sentry (error only)5,72467%5,912-3%
POST Baseline1,061-1,208-12%
POST With Sentry49447%603-18%
POST With Sentry (error only)95090%1,058-10%
MYSQL Baseline3,073-3,231-5%
MYSQL With Sentry33311%506-34%
MYSQL With Sentry (error only)2,44980%2,684-9%

View base workflow run

@stephanie-anderson
stephanie-anderson enabled auto-merge (squash) March 30, 2026 10:15
@stephanie-anderson
stephanie-anderson merged commit fff880e into developMar 30, 2026
72 checks passed
@stephanie-anderson
stephanie-anderson deleted the fix/update-validate-pr-action branch March 30, 2026 10:19
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@stephanie-anderson@dingsdax@Lms24@chargome