workflow: minor tweaks for release script (#10220)

- `pnpm run test` instead of `pnpm test run`
- also need `--no-git-checks` when drying run
- fix spelling of `remote`

diff --git a/scripts/release.js b/scripts/release.js
index 6a962e07adfdbb8931d7514197271848234c09a5..7ee29185a49da8e4ba44a24318981ebfbf3bb041 100644 (file)
--- a/scripts/release.js
+++ b/scripts/release.js
@@ -114,7 +114,7 @@ async function main() {
   if (!(await isInSyncWithRemote())) {
     return
   } else {
-    console.log(`${pico.green(`✓`)} commit is up-to-date with rmeote.\n`)
+    console.log(`${pico.green(`✓`)} commit is up-to-date with remote.\n`)
   }
 
   let targetVersion = args._[0]
@@ -239,7 +239,7 @@ async function main() {
   if (!skipTests) {
     step('\nRunning tests...')
     if (!isDryRun) {
-      await run('pnpm', ['test', 'run'])
+      await run('pnpm', ['run', 'test'])
     } else {
       console.log(`Skipped (dry run)`)
     }
@@ -307,7 +307,7 @@ async function main() {
   if (isDryRun) {
     additionalPublishFlags.push('--dry-run')
   }
-  if (skipGit) {
+  if (isDryRun || skipGit) {
     additionalPublishFlags.push('--no-git-checks')
   }
   // bypass the pnpm --publish-branch restriction which isn't too useful to us