Fix CI failures: remove Test-ModuleManifest, fix lint warnings and table alignment

- Remove Test-ModuleManifest call that fails when RequiredModules are not
  yet installed (dependencies are resolved in a later step)
- Add SuppressMessage attributes for prNumber and prHeadRef variables
  assigned in LogGroup blocks but used in subsequent blocks
- Break long error message line to satisfy PSAvoidLongLines rule
- Fix README table pipe alignment (MD060) by shortening ModulePath
  description to fit within the aligned column width

Author: MariusStorhaug

README.md

@@ -39,14 +39,14 @@ This makes the tested artifact identical to the published artifact (see
 
 ## Inputs
 
-| Name                       | Description                                                                                | Required | Default          |
-| -------------------------- | ------------------------------------------------------------------------------------------ | -------- | ---------------- |
-| `Name`                     | Name of the module. Defaults to the repository name.                                       | No       | Repository name  |
-| `ModulePath`               | Path to the folder that contains the `<Name>/` module subdirectory (e.g. `outputs/module` must contain `outputs/module/<Name>/`). | No       | `outputs/module` |
-| `APIKey`                   | PowerShell Gallery API key.                                                                | Yes      |                  |
-| `AutoCleanup`              | Delete prerelease tags matching the PR branch after a stable release.                      | No       | `true`           |
-| `WhatIf`                   | Log the changes that would be made without publishing, creating, or deleting anything.    | No       | `false`          |
-| `WorkingDirectory`         | The working directory where the script will run from.                                      | No       | `.`              |
-| `UsePRTitleAsReleaseName`  | Use the PR title as the release name (otherwise the version string is used).               | No       | `false`          |
-| `UsePRBodyAsReleaseNotes`  | Use the PR body as the release notes (otherwise `--generate-notes` is used).               | No       | `true`           |
-| `UsePRTitleAsNotesHeading` | Prefix the release notes with the PR title as an H1 heading linking to the PR.             | No       | `true`           |
+| Name                       | Description                                                                             | Required | Default          |
+| -------------------------- | --------------------------------------------------------------------------------------- | -------- | ---------------- |
+| `Name`                     | Name of the module. Defaults to the repository name.                                    | No       | Repository name  |
+| `ModulePath`               | Path to the folder containing the `<Name>/` module subdirectory from Build-PSModule.    | No       | `outputs/module` |
+| `APIKey`                   | PowerShell Gallery API key.                                                             | Yes      |                  |
+| `AutoCleanup`              | Delete prerelease tags matching the PR branch after a stable release.                   | No       | `true`           |
+| `WhatIf`                   | Log the changes that would be made without publishing, creating, or deleting anything.  | No       | `false`          |
+| `WorkingDirectory`         | The working directory where the script will run from.                                   | No       | `.`              |
+| `UsePRTitleAsReleaseName`  | Use the PR title as the release name (otherwise the version string is used).            | No       | `false`          |
+| `UsePRBodyAsReleaseNotes`  | Use the PR body as the release notes (otherwise `--generate-notes` is used).            | No       | `true`           |
+| `UsePRTitleAsNotesHeading` | Prefix the release notes with the PR title as an H1 heading linking to the PR.          | No       | `true`           |

src/publish.ps1

@@ -14,6 +14,14 @@
     'PSUseDeclaredVarsMoreThanAssignments', 'usePRTitleAsNotesHeading',
     Justification = 'Variable is used in script blocks.'
 )]
+[Diagnostics.CodeAnalysis.SuppressMessageAttribute(
+    'PSUseDeclaredVarsMoreThanAssignments', 'prNumber',
+    Justification = 'Variable is used in script blocks.'
+)]
+[Diagnostics.CodeAnalysis.SuppressMessageAttribute(
+    'PSUseDeclaredVarsMoreThanAssignments', 'prHeadRef',
+    Justification = 'Variable is used in script blocks.'
+)]
 [CmdletBinding()]
 param()
 
@@ -70,11 +78,11 @@ LogGroup 'Resolve version from manifest' {
 
     Show-FileContent -Path $manifestFilePath
 
-    $manifest = Test-ModuleManifest -Path $manifestFilePath -ErrorAction Stop
     $manifestData = Import-PowerShellDataFile -Path $manifestFilePath
     $moduleVersion = $manifestData.ModuleVersion
     if (-not ($moduleVersion -match '^\d+\.\d+\.\d+$')) {
-        Write-Error "ModuleVersion [$moduleVersion] must be in Major.Minor.Patch format. Ensure Build-PSModule has stamped the artifact with a final version."
+        Write-Error ("ModuleVersion [$moduleVersion] must be in Major.Minor.Patch format. " +
+            'Ensure Build-PSModule has stamped the artifact with a final version.')
         exit 1
     }
     $prerelease = $manifestData.PrivateData.PSData.Prerelease