chore: todos

Author: posva

__tests__/utils.ts

@@ -103,7 +103,7 @@ export function createDom(options?: ConstructorOptions) {
     }
   )
 
-  // @ts-ignore: TODO: remove this line then?
+  // @ts-ignore: needed for jsdom
   global.window = dom.window
   global.___location = dom.window.___location
   global.history = dom.window.history

src/matcher/index.ts

@@ -241,7 +241,7 @@ export function createRouterMatcher(
       // this also allows the user to control the encoding
       path = ___location.path
 
-      if (__DEV__ && path[0] !== '/') {
+      if (__DEV__ && !path.startsWith('/')) {
         warn(
           `The Matcher cannot resolve relative paths but received "${path}". Unless you directly called \`matcher.resolve("${path}")\`, this is probably a bug in vue-router. Please open an issue at https://new-issue.vuejs.org/?repo=vuejs/vue-router-next.`
         )
@@ -252,6 +252,7 @@ export function createRouterMatcher(
 
       if (matcher) {
         // TODO: dev warning of unused params if provided
+        // we know the matcher works because we tested the regexp
         params = matcher.parse(path)!
         name = matcher.record.name
       }