Always log_info when we broadcast a transaction, including the txid

[TheBlueMatt]

No description provided.

[codecov]

Codecov Report

Merging #914 (1c948ec) into main (c9b12e1) will increase coverage by 0.05%.
The diff coverage is 92.85%.

❗ Current head 1c948ec differs from pull request most recent head 71d640a. Consider uploading reports for the commit 71d640a to get more accurate results

@@            Coverage Diff             @@
##             main     #914      +/-   ##
==========================================
+ Coverage   90.51%   90.57%   +0.05%     
==========================================
  Files          59       59              
  Lines       29746    29751       +5     
==========================================
+ Hits        26926    26946      +20     
+ Misses       2820     2805      -15     

Impacted Files	Coverage Δ
lightning/src/util/macro_logger.rs	88.88% <85.71%> (+1.79%)	⬆️
lightning/src/chain/channelmonitor.rs	95.46% <100.00%> (+<0.01%)	⬆️
lightning/src/ln/channelmanager.rs	84.03% <100.00%> (+0.05%)	⬆️
lightning/src/ln/onchaintx.rs	87.47% <100.00%> (+0.20%)	⬆️
lightning/src/ln/functional_tests.rs	97.04% <0.00%> (+0.20%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c9b12e1...71d640a. Read the comment docs.

[valentinewallace]

LGTM, just some nits

[TheBlueMatt]

Squashed, only change from the original PR is the minor typo:

-				log_info!(logger, "Broadcasting onchaing {}", log_tx!(bump_tx));
+				log_info!(logger, "Broadcasting onchain {}", log_tx!(bump_tx));