-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
[Templates] Plugin deps & Custom.d.ts #15798
Conversation
Codecov ReportBase: 58.40% // Head: 58.41% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #15798 +/- ##
=======================================
Coverage 58.40% 58.41%
=======================================
Files 1545 1545
Lines 38196 38199 +3
Branches 7512 7510 -2
=======================================
+ Hits 22309 22314 +5
+ Misses 13594 13592 -2
Partials 2293 2293
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
packages/generators/generators/lib/files/ts/plugin/tsconfig.json
Outdated
Show resolved
Hide resolved
packages/generators/generators/lib/templates/ts/plugin-package.json.hbs
Outdated
Show resolved
Hide resolved
Co-authored-by: markkaylor <mark.kaylor@strapi.io>
….json.hbs Co-authored-by: markkaylor <mark.kaylor@strapi.io>
@joshuaellis Unfortunately I don't feel comfortable to review this PR. Generators are owned by @markkaylor and @remidej and I believe they should be the ones reviewing this. |
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.
Should we remove |
@joshuaellis I'd love that. |
So because we don't have types for the I've also removed |
Size Change: 0 B Total Size: 1.54 MB ℹ️ View Unchanged
|
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.
Looks good to me 👍
I couldn't see any documentation on the site for this though... so i'm not sure how we communicate this change?
I guess maybe we could add a small comment in the release notes?
chore: fix scripts chore: fix dev to be develop Update examples/getstarted/src/index.js Update examples/getstarted/src/index.js Co-Authored-By: markkaylor <mark.kaylor@strapi.io>
c6c395d
to
29761db
Compare
What does it do?
custom.d.ts
file for Typescript plugins so it doesn't fail with our own packagesWhy is it needed?
Related issue(s)/PR(s)