feat: Notify Users about out of date pack#590
Conversation
kristiankunc
left a comment
kristiankunc
left a comment
There was a problem hiding this comment.
I'd honestly love to see at least partial test coverage on this.
|
I'll write some in a bit |
|
The copilot comments about the files also look sensible, we can just throw an error instantly (which can be surpressed as I assume we don't want to enforce it 100%) + the rest are some nitpicks |
kristiankunc
changed the title
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@BenWalker01 still planning on pushing this? |
|
Yeah. Will try fix the outstanding comments this evening! |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
BenWalker01
force-pushed
the
ukcp-update-notifs
branch
from
e107a61 to
cfea261
Compare
BenWalker01
changed the title
|
🎉 This PR is included in version 5.18.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
3 participants
Fixes #571