ci: unify github actions for merge and pull request

There's no need to do anything different and last merge showed it's not good when the
actions drift apart. PR coverage was ok, but merge coverage is abysmal since the action
fell behind.

And while we're at it, remove the `// ignore.coverage` comment from lines where it has no
effect.

.github/workflows/pr_checks.yaml → .github/workflows/checks.yaml

@@ -1,7 +1,10 @@
 name: Check
 
 on:
-  - pull_request
+  pull_request:
+  push:
+    branches:
+      - main
 
 jobs:
   lint-commit:

client/root.go

@@ -79,7 +79,7 @@ with the given id.` +
 			err = ui.App.Run()
 			if err != nil { // ignore.coverage
 				log.Fatalf("Could not do stuff: %v", err) // ignore.coverage
-			} // ignore.coverage
+			}
 		} else {
 			fmt.Println("Running in test mode - no tui, no ws")
 		}
@@ -101,12 +101,12 @@ func getWsURL() string {
 		res, err := client.Get(roomURL) //nolint:bodyclose // its in the defer below
 		if err != nil {                 // ignore.coverage
 			log.Fatalf("error making http request: %s\n", err) // ignore.coverage
-		} // ignore.coverage
+		}
 		defer func(Body io.ReadCloser) {
 			err := Body.Close()
 			if err != nil { // ignore.coverage
 				log.Fatalf("error closing body: %s\n", err) // ignore.coverage
-			} // ignore.coverage
+			}
 		}(res.Body)
 		var location string
 		if res.StatusCode == http.StatusTemporaryRedirect {

client/tui.go

@@ -110,7 +110,7 @@ func (tui *TUI) createUsersTable() *tview.Flex {
 	_, err := usersText.Write([]byte(usernames))
 	if err != nil { // ignore.coverage
 		log.Fatalf("Could not write to text: %v", err) // ignore.coverage
-	} // ignore.coverage
+	}
 
 	cardValuesText := tview.NewTextView().
 		SetWordWrap(false)
@@ -122,7 +122,7 @@ func (tui *TUI) createUsersTable() *tview.Flex {
 	_, err = cardValuesText.Write([]byte(cardValues))
 	if err != nil { // ignore.coverage
 		log.Fatalf("Could not write to text: %v", err) // ignore.coverage
-	} // ignore.coverage
+	}
 
 	usersTable := tview.NewFlex().
 		AddItem(usersText, 0, 3, false).
@@ -138,7 +138,7 @@ func (tui *TUI) createUsersTable() *tview.Flex {
 	_, err = averageText.Write([]byte(tui.Room.Average))
 	if err != nil { // ignore.coverage
 		log.Fatalf("Could not write to text: %v", err) // ignore.coverage
-	} // ignore.coverage
+	}
 
 	tableWithAverage := tview.NewFlex().
 		AddItem(nil, 0, 3, false).