docs: updates legacy packages with info regarding their removal#2519
docs: updates legacy packages with info regarding their removal#2519lukegalbraithrussell merged 2 commits intopfrom
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## p #2519 +/- ##
=======================================
Coverage 93.17% 93.17%
=======================================
Files 40 40
Lines 11384 11384
Branches 726 726
=======================================
Hits 10607 10607
Misses 765 765
Partials 12 12
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
zimeg
left a comment
There was a problem hiding this comment.
@lukegalbraithrussell Thanks so much for keeping references around! I'd imagine it's frustrating to find a missing package otherwise...
LGTM and no blockers from me, but I notice we also have a mention in our README and I wasn't sure if this should be removed perhaps? Or changed similar? Or left as is! 📚 ✨
Lines 33 to 37 in 59df200
|
@zimeg I removed the README callout! imo readmes should reflect current state |
zimeg
left a comment
There was a problem hiding this comment.
🏆 @lukegalbraithrussell LGTMs all around!
I'm curious if we can remove the "deprecated packages" section of the sidebar too, perhaps considering the suggestion to @slack/bolt as part of "migration" instead? No blocker since I'm not too familiar with these page rankings 🤓
Summary
👋
Requirements