Skip to content

feat: add Svelte CI anomalies to download-anomalies.data.ts#1934

Merged
graphieros merged 3 commits intonpmx-dev:mainfrom
samal-rasmussen:svelte-anomalies
Mar 5, 2026
Merged

feat: add Svelte CI anomalies to download-anomalies.data.ts#1934
graphieros merged 3 commits intonpmx-dev:mainfrom
samal-rasmussen:svelte-anomalies

Conversation

@samal-rasmussen
Copy link
Contributor

Add Svelte CI anomalies to download-anomalies.data.ts

@vercel
Copy link

vercel bot commented Mar 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Mar 5, 2026 5:19am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Mar 5, 2026 5:19am
npmx-lunaria Ignored Ignored Mar 5, 2026 5:19am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 20555381-0e19-48c5-b9dc-3bb89c8dd887

📥 Commits

Reviewing files that changed from the base of the PR and between 6f460e6 and f33034a.

📒 Files selected for processing (1)
  • app/utils/download-anomalies.data.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • app/utils/download-anomalies.data.ts

📝 Walkthrough

Walkthrough

The pull request updates app/utils/download-anomalies.data.ts by adding three new entries for packageName 'svelte' to the DOWNLOAD_ANOMALIES array, each with specific start/end dates and weeklyDownloads values. A NOTE comment block was also added inside the array to indicate where to add future entries and to explain changes. No logic, exported interfaces, or public signatures were altered.

Possibly related PRs

Suggested reviewers

  • alexdln
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description accurately relates to the changeset, which adds three new Svelte anomaly entries to the download-anomalies.data.ts file.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@samal-rasmussen samal-rasmussen changed the title Add Svelte CI anomalies to download-anomalies.data.ts feat: Add Svelte CI anomalies to download-anomalies.data.ts Mar 4, 2026
@samal-rasmussen samal-rasmussen changed the title feat: Add Svelte CI anomalies to download-anomalies.data.ts feat: add Svelte CI anomalies to download-anomalies.data.ts Mar 4, 2026
@ghostdevv ghostdevv requested a review from graphieros March 4, 2026 23:47
Copy link
Contributor

@graphieros graphieros left a comment

Choose a reason for hiding this comment

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

@samal-rasmussen

Thank you for your PR :)
Would this peak also require some correction ?

image

Also, I wonder if the correction for this sequence should start slightly before:

image

Wdyt ?

@samal-rasmussen
Copy link
Contributor Author

samal-rasmussen commented Mar 5, 2026

Thank you for your PR :) Would this peak also require some correction ?

I already have that one in the pr.

Also, I wonder if the correction for this sequence should start slightly before:

I believe it starts and ends exactly right as I've already set it. I used the csv download to get the precise dates for the data:

2022-11-09,73407
2022-11-10,76389
2022-11-11,61405
2022-11-12,28670
2022-11-13,31092
2022-11-14,74297
2022-11-15,75233
2022-11-16,2075741
2022-11-17,3631844
2022-11-18,3832375
2022-11-19,3754596
2022-11-20,3749193
2022-11-21,3649561
2022-11-22,4167758
2022-11-23,5035961
2022-11-24,3952970
2022-11-25,3927307
2022-11-26,3945833
2022-11-27,3980243
2022-11-28,2378449
2022-11-29,155912
2022-11-30,69524
2022-12-01,72935
2022-12-02,63625
2022-12-03,30534
2022-12-04,31255
2022-12-05,71156
2022-12-06,74600
2022-12-07,74681

2023-06-14,97342
2023-06-15,90999
2023-06-16,83554
2023-06-17,38247
2023-06-18,39071
2023-06-19,107491
2023-06-20,1875609
2023-06-21,8613971
2023-06-22,112432
2023-06-23,91697
2023-06-24,54947
2023-06-25,46621
2023-06-26,106338
2023-06-27,106723

2023-11-14,140285
2023-11-15,133604
2023-11-16,144798
2023-11-17,142315
2023-11-18,59611
2023-11-19,585688
2023-11-20,390406
2023-11-21,150680
2023-11-22,133466
2023-11-23,113607
2023-11-24,100603
2023-11-25,54194
2023-11-26,59743
2023-11-27,136117

I've been very intentional not to exclude data that is plausible. I've set the data ranges as tight as possible, as to only exclude data that is clearly way out of the range seen before of after. I've only excluded data that is a sudden doubling or tripling or more from the days around it.

@graphieros graphieros self-requested a review March 5, 2026 11:26
Copy link
Contributor

@graphieros graphieros left a comment

Choose a reason for hiding this comment

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

Thank you for your explanations!

@graphieros graphieros added this pull request to the merge queue Mar 5, 2026
Merged via the queue into npmx-dev:main with commit d10e3c3 Mar 5, 2026
20 checks passed
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

Thanks for your first contribution, @samal-rasmussen! 🥳

We'd love to welcome you to the npmx community. Come and say hi on Discord! And once you've joined, visit npmx.wamellow.com to claim the contributor role.

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