GitLab

This patch removes the 'exit 1' status code when the update check
finishes or is ended with CTRL+C.

Fixes #10033

Fixes #8427

Fixes #9701

* refactor(install.sh): fix static analysis warnings

Clear all warnings and errors raised by shellcheck.net static analysis.

- Replace non-POSIX shell use of `$OSTYPE` by POSIX compliant check on `uname -o`.
- Move variables out of`printf` format string.
- Refactor/simplify string formatters for error and underline.
- Fix expansion of arguments to a single string `$*` rather than individual elements `$@` within the error and underline formatters.

* fix(uname): non-posix -o option

* fix(install.sh): non portable which

Replaced non-portable `which zsh` by portable `command -v zsh`

Fixes #9629
Fixes #9700

* Don't error on upgrade no-op

No error code is required for a non failure scenario.

* Manually check whether changes were pulled in `omz update`
Co-authored-by: Marc Cornellà <hello@mcornella.com>

The GitHub UI might not remove CR characters from commit description forms
filled from a Windows browser.

Fixes #9509

This also allows the option to put extra paragraphs after the BREAKING CHANGE
message while properly displaying the breaking change message. Useful, for
example, to add signed-off or co-authored lines.

Fixes #9505

Also fixed a call to a defunct display:type-header function in displaying
breaking changes.

Fixes #9495

I used _ which is a convention in other languages, but in shell scripting
$_ is a special variable set by the shell, and in Zsh versions older than
5.0.6 it complains for being a `read-only variable`.

Fixes #9482

Also prefix all formatting functions with fmt_

Related: #9427

Fixes #9237

Co-authored-by: Marc Cornellà <marc.cornella@live.com>

* Suppress the problematic trap output in check_upg

The newly added trap, in systems where `rm` is aliased to `rm="rm -v"`,
shows a message stating that "update.lock" has been removed each time `zsh` is called.
I simply suppressed it with directing the output to `/dev/null`.

* Use `command` instead of >/dev/null to suppress

Changing 'your' to 'you'

Co-authored-by: Marc Cornellà <marc.cornella@live.com>

Fixes #8732