-
-
Notifications
You must be signed in to change notification settings - Fork 8.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
Feature/main template type support #7116
Conversation
lib/MainTemplate.js
Outdated
/** | ||
* | ||
* @param {any} options render manifest options | ||
* @returns {any[]} returns render nanifest |
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.
@TheLarkInn nanifest 😂
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.
Lmaooooo nanifest seems cooler though 😂
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.
Willfix
…l signatures for chunk and main templates
Rerunning CI with new TypeScript nightly now in master (which this was dependent on) |
Merging now that type errors and lint failures are gone, and with @ooflorent approval. |
Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon. |
What kind of change does this PR introduce?
Typing JSDocs
Did you add tests for your changes?
N/A
If relevant, link to documentation update:
N/A
Summary
Add's Type support via JSDocs for MainTemplate class.
Does this PR introduce a breaking change?
No
Other information
This PR although may include TS nightly change in package.json actually relies on official version bump PR #7115