search

webpack / enhanced-resolve

Offers an async require.resolve function. It's highly configurable.
MIT License
916 stars 186 forks source link

refactor: types #381

Closed alexander-akait closed 1 year ago

alexander-akait commented 1 year ago

Add more types +strict mode, also update typescript and add commands to get coverage + fix async tests + improve ci

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 87.75% and project coverage change: -0.06 :warning:

Comparison is base (d5cb40e) 92.90% compared to head (76b2c9e) 92.85%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #381 +/- ## ========================================== - Coverage 92.90% 92.85% -0.06% ========================================== Files 43 43 Lines 2029 2042 +13 Branches 598 598 ========================================== + Hits 1885 1896 +11 - Misses 116 118 +2 Partials 28 28 ``` | [Impacted Files](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack) | Coverage Δ | | |---|---|---| | [lib/AliasPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL0FsaWFzUGx1Z2luLmpz) | `100.00% <ø> (ø)` | | | [lib/AppendPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL0FwcGVuZFBsdWdpbi5qcw==) | `100.00% <ø> (ø)` | | | [lib/DescriptionFilePlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL0Rlc2NyaXB0aW9uRmlsZVBsdWdpbi5qcw==) | `100.00% <ø> (ø)` | | | [lib/ExtensionAliasPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL0V4dGVuc2lvbkFsaWFzUGx1Z2luLmpz) | `97.22% <ø> (ø)` | | | [lib/LogInfoPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL0xvZ0luZm9QbHVnaW4uanM=) | `0.00% <ø> (ø)` | | | [lib/ModulesInRootPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL01vZHVsZXNJblJvb3RQbHVnaW4uanM=) | `100.00% <ø> (ø)` | | | [lib/ParsePlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL1BhcnNlUGx1Z2luLmpz) | `96.77% <ø> (ø)` | | | [lib/RestrictionsPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL1Jlc3RyaWN0aW9uc1BsdWdpbi5qcw==) | `100.00% <ø> (ø)` | | | [lib/RootsPlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL1Jvb3RzUGx1Z2luLmpz) | `100.00% <ø> (ø)` | | | [lib/SelfReferencePlugin.js](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack#diff-bGliL1NlbGZSZWZlcmVuY2VQbHVnaW4uanM=) | `96.29% <ø> (ø)` | | | ... and [27 more](https://app.codecov.io/gh/webpack/enhanced-resolve/pull/381?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=webpack) | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.