Repository navigation
fix: changelog - check release tag before title for deciding auto scroll. - #3305
Conversation
Thanks for opening this pull request! 🎉We really appreciate you taking the time to contribute, @WilcoSp. A maintainer will take a look as soon as they can. In the meantime, please make sure that:
If anything needs adjusting we'll leave comments here. Thanks again! |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to Release auto-scroll now matches tags as well as titles, while preserving date-based navigation when neither matches. No concrete issue has been established, so the change appears ready to merge subject to normal checks. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review details
🚥 Pre-merge checks | ✅ 4
✨ Finishing Touches
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. Comment |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
…oll. (#3305) Co-authored-by: Alec Lloyd Probert <55991794+graphieros@users.noreply.github.com>
🔗 Linked issue
None
🧭 Context & 📚 Description
Before the auto scrolling at package changelog only used the release title for deciding where to auto scroll to, this pr adds checking the release tag and makes it the first to check.
This is most noticeable when viewing the changelogs for pnpm in the url hash.
here is an example with pnpm: