Skip to content

chore: Add aws/gcp serverless and deno to craft release registry target#12562

Merged
AbhiPrasad merged 1 commit intodevelopfrom
chore-add-new-package-release-registry
Jun 24, 2024
Merged

chore: Add aws/gcp serverless and deno to craft release registry target#12562
AbhiPrasad merged 1 commit intodevelopfrom
chore-add-new-package-release-registry

Conversation

@AbhiPrasad
Copy link
Copy Markdown
Contributor

ref getsentry/sentry-release-registry#161 and #12442

Adds these packages to the craft config so they can be updated in the release registry.

@AbhiPrasad AbhiPrasad self-assigned this Jun 19, 2024
@AbhiPrasad AbhiPrasad requested review from a team, chargome and nicohrubec and removed request for a team June 19, 2024 14:20
@github-actions
Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
@sentry/browser 22.22 KB (0%)
@sentry/browser (incl. Tracing) 33.27 KB (0%)
@sentry/browser (incl. Tracing, Replay) 69.05 KB (0%)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 62.37 KB (0%)
@sentry/browser (incl. Tracing, Replay with Canvas) 73.11 KB (0%)
@sentry/browser (incl. Tracing, Replay, Feedback) 85.27 KB (-0.01% 🔽)
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 87.09 KB (-0.01% 🔽)
@sentry/browser (incl. metrics) 26.42 KB (0%)
@sentry/browser (incl. Feedback) 38.42 KB (+0.01% 🔺)
@sentry/browser (incl. sendFeedback) 26.85 KB (0%)
@sentry/browser (incl. FeedbackAsync) 31.42 KB (+0.01% 🔺)
@sentry/react 24.96 KB (+0.01% 🔺)
@sentry/react (incl. Tracing) 36.32 KB (+0.01% 🔺)
@sentry/vue 26.23 KB (0%)
@sentry/vue (incl. Tracing) 35.13 KB (0%)
@sentry/svelte 22.35 KB (0%)
CDN Bundle 23.41 KB (0%)
CDN Bundle (incl. Tracing) 34.95 KB (0%)
CDN Bundle (incl. Tracing, Replay) 69.08 KB (0%)
CDN Bundle (incl. Tracing, Replay, Feedback) 74.25 KB (0%)
CDN Bundle - uncompressed 68.76 KB (+0.01% 🔺)
CDN Bundle (incl. Tracing) - uncompressed 103.41 KB (+0.01% 🔺)
CDN Bundle (incl. Tracing, Replay) - uncompressed 213.88 KB (+0.01% 🔺)
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 226.47 KB (+0.01% 🔺)
@sentry/nextjs (client) 36.2 KB (0%)
@sentry/sveltekit (client) 33.9 KB (+0.01% 🔺)
@sentry/node 112.94 KB (+0.29% 🔺)
@sentry/node - without tracing 90.35 KB (+0.36% 🔺)
@sentry/aws-serverless 99.44 KB (+0.32% 🔺)

@AbhiPrasad AbhiPrasad requested review from lforst and mydea June 19, 2024 15:05
onlyIfPresent: /^sentry-google-cloud-serverless-\d.*\.tgz$/
'npm:@sentry/nextjs':
onlyIfPresent: /^sentry-nextjs-\d.*\.tgz$/
'npm:@sentry/node':
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

h: I think you ordered this by name incorrectly, the order here is important! we need to have packages first that others depend on, e.g. react and node need to be higher in order before nextjs?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this ordering matter for the release registry? This only runs after we publish on npm too - so we should be good right?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahh ok, you are right of course. This is all a bit confusing! It confused me a bit because it says "npm:..."

@AbhiPrasad AbhiPrasad merged commit 7a9151b into develop Jun 24, 2024
@AbhiPrasad AbhiPrasad deleted the chore-add-new-package-release-registry branch June 24, 2024 13:15
Sign up for free to 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.

3 participants