Fix c8 ignore comment formatting by adding periods

Ensure all c8 ignore comments end with periods for consistency
with established code style guidelines.

Author: jdalton

src/decode.ts

@@ -10,7 +10,7 @@ function decodePurlComponent(comp: string, encodedComponent: string): string {
   try {
     return decodeComponent(encodedComponent)
   } catch {
-    /* c8 ignore next -- Intentionally empty. Invalid encoding will throw below. */
+    /* c8 ignore next -- Intentionally empty, invalid encoding will throw below. */
   }
   throw new PurlError(`unable to decode "${comp}" component`)
 }

src/package-url.ts

@@ -308,7 +308,7 @@ class PackageURL {
     // Split the remainder once from left on ':'.
     const colonIndex = purlStr.indexOf(':')
     // Use WHATWG URL to split up the purl string.
-    /* c8 ignore next 3 -- Comment lines don't need coverage */
+    /* c8 ignore next 3 -- Comment lines don't need coverage. */
     //   - Split the purl string once from right on '#'
     //   - Split the remainder once from right on '?'
     //   - Split the remainder once from left on ':'
@@ -324,7 +324,7 @@ class PackageURL {
         const afterColon = purlStr.slice(colonIndex + 1)
         const trimmedAfterColon = trimLeadingSlashes(afterColon)
         url = new URL(`${beforeColon}:${trimmedAfterColon}`)
-        /* c8 ignore next 4 -- V8 coverage sees multiple branch paths in ternary that can't all be tested */
+        /* c8 ignore next 4 -- V8 coverage sees multiple branch paths in ternary that can't all be tested. */
         maybeUrlWithAuth =
           afterColon.length === trimmedAfterColon.length
             ? url
@@ -336,10 +336,10 @@ class PackageURL {
       }
     }
     // The scheme is a constant with the value "pkg".
-    /* c8 ignore next -- Tested: colonIndex === -1 (url undefined) case, but V8 can't see both branches */
+    /* c8 ignore next -- Tested: colonIndex === -1 (url undefined) case, but V8 can't see both branches. */
     if (url?.protocol !== 'pkg:') {
       throw new PurlError('missing required "pkg" scheme component')
-      /* c8 ignore next -- Unreachable code after throw */
+      /* c8 ignore next -- Unreachable code after throw. */
     }
     // A purl must NOT contain a URL Authority i.e. there is no support for
     // username, password, host and port components.
@@ -362,7 +362,7 @@ class PackageURL {
 
     let rawVersion: string | undefined
     // Both branches of this ternary are tested, but V8 reports phantom branch combinations
-    /* c8 ignore start -- npm vs non-npm path logic both tested but V8 sees extra branches */
+    /* c8 ignore start -- npm vs non-npm path logic both tested but V8 sees extra branches. */
     let atSignIndex =
       rawType === 'npm'
         ? // Deviate from the specification to handle a special npm purl type case for
@@ -419,7 +419,7 @@ class PackageURL {
         const value = decodePurlComponent('qualifiers', pairs.at(1) ?? '')
         // Use URLSearchParams#append to preserve plus signs.
         // https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams#preserving_plus_signs
-        /* c8 ignore next -- URLSearchParams.append has internal V8 branches we can't control */
+        /* c8 ignore next -- URLSearchParams.append has internal V8 branches we can't control. */
         searchParams.append(pairs[0]!, value)
       }
       // Split the remainder once from right on '?'.

src/purl-type.ts

@@ -408,7 +408,7 @@ const PurlType = createHelpersNamespaceObject(
             if (throws) {
               // Tested: validation returns false in non-throw mode
               // V8 coverage can't see both throw and return false paths in same test
-              /* c8 ignore next 3 -- Throw path tested separately from return false path */
+              /* c8 ignore next 3 -- Throw path tested separately from return false path. */
               throw new PurlError(
                 `npm "namespace" and "name" components can not collectively be more than 214 characters`,
               )
@@ -435,7 +435,7 @@ const PurlType = createHelpersNamespaceObject(
             if (throws) {
               // Tested: validation returns false in non-throw mode
               // V8 coverage can't see both throw and return false paths in same test
-              /* c8 ignore next 3 -- Throw path tested separately from return false path */
+              /* c8 ignore next 3 -- Throw path tested separately from return false path. */
               throw new PurlError(
                 'npm "name" component can not be a core module name',
               )
@@ -467,7 +467,7 @@ const PurlType = createHelpersNamespaceObject(
               if (throws) {
                 // Tested: validation returns false in non-throw mode
                 // V8 coverage can't see both throw and return false paths in same test
-                /* c8 ignore next 3 -- Throw path tested separately from return false path */
+                /* c8 ignore next 3 -- Throw path tested separately from return false path. */
                 throw new PurlError(
                   'pub "name" component may only contain [a-z0-9_] characters'
                 )

src/validate.ts

@@ -184,7 +184,7 @@ function validateType(type: unknown, throws: boolean): boolean {
     ) {
       if (throws) {
         throw new PurlError(`type "${type}" contains an illegal character`)
-        /* c8 ignore next -- Unreachable code after throw */
+        /* c8 ignore next -- Unreachable code after throw. */
       }
       return false
     }