search

dvajs / dva

🌱 React and redux based, lightweight and elm-style framework. (Inspired by elm and choo)
https://dvajs.com/
MIT License
16.24k stars 3.17k forks source link

fix(dva): typing of param in interface 'Router' #2459

Closed WavinFlag closed 2 years ago

WavinFlag commented 3 years ago

The param api is non-null, according to the source code: https://github.com/dvajs/dva/blob/3eaee309ede9cf1e255326150ee2210bd04c1abf/packages/dva/src/index.js#L96

Checklist
Description of change
codecov-commenter commented 3 years ago

Codecov Report

Merging #2459 (d1ed7ed) into master (3eaee30) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2459   +/-   ##
=======================================
  Coverage   85.90%   85.90%           
=======================================
  Files          18       18           
  Lines         518      518           
  Branches      114      114           
=======================================
  Hits          445      445           
  Misses         59       59           
  Partials       14       14

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3eaee30...d1ed7ed. Read the comment docs.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.