Merge rust-bitcoin#4180: Fix the release script that checks for TBD

fe685b8 Fix the release script that checks for TBD (Jamil Lambert, PhD)

Pull request description:

  `release.sh` is missing `units`.

  Add `units` to the list of crates to check.

  Reorder the crates alphabetically.

  Fix rust-bitcoin#4163

ACKs for top commit:
  tcharding:
    ACK fe685b8
  Kixunil:
    ACK fe685b8
  apoelstra:
    ACK fe685b8; successfully ran local tests

Tree-SHA512: 0b1ac9610e547f895902ce5d73aa7d2fcdcb319bded3f78e490d3288f177a79c5e2ee354e0481e128dbef0f7b134737422272a615d3bb0fef4f3bd05e5ba4c0a

Author: apoelstra

contrib/release.sh

@@ -6,7 +6,7 @@
 set -euox pipefail
 
 main () {
-    for crate in "internals" "hashes" "bitcoin"; do
+    for crate in "bitcoin" "hashes" "internals" "units"; do
         if release_changes $crate; then
             echo "$crate has changes implying this is a release PR, checking if we can publish ..."
 
@@ -35,12 +35,14 @@ release_changes() {
 # We use `set -e` so this will fail the script if the dry-run fails.
 publish_dry_run() {
     local crate=$1
-    if [ "$crate" == "hashes" ]; then
+    if [ "$crate" == "bitcoin" ]; then
+        cargo publish -p "bitcoin" --dry-run
+    elif [ "$crate" == "hashes" ]; then
         cargo publish -p "bitcoin_hashes" --dry-run
     elif [ "$crate" == "internals" ]; then
         cargo publish -p "bitcoin-internals" --dry-run
-    elif [ "$crate" == "bitcoin" ]; then
-        cargo publish -p "bitcoin" --dry-run
+    elif [ "$crate" == "units" ]; then
+        cargo publish -p "bitcoin-units" --dry-run
     fi
 }