New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cwray-tech/update plugin descriptions #8921
cwray-tech/update plugin descriptions #8921
Conversation
The marketplace area in the cms shows this text, so I thought this would be a little more helpful than the default text. Signed-off-by: Christopher Wray <53663762+cwray-tech@users.noreply.github.com>
@alexandrebodin Love it if you could check this out and merge before next update haha. |
Codecov Report
@@ Coverage Diff @@
## master #8921 +/- ##
==========================================
+ Coverage 33.47% 34.56% +1.08%
==========================================
Files 1249 1307 +58
Lines 13747 14355 +608
Branches 1367 1419 +52
==========================================
+ Hits 4602 4962 +360
- Misses 8255 8484 +229
- Partials 890 909 +19
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Actually, Scratch that? some of the linting tests didn't pass. I will keep this fork active this time, so no pressure to quickly merge! @alexandrebodin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the improvement :)
What does it do?
This is just adding some simple plugin descriptions as originally discussed on this pr: #8254
I simply updated the descriptions in the package.json files.
Why is it needed?
This is needed because it will be more accurate descriptions of the plugins in places where these descriptions are used.