RuntimeTransaction: Minor Cleanup

[apfitzge]

Problem

Following up on some minor thoughts I had from #33471.

Summary of Changes

• Remove simple comments which repeat field names
• TransactionMeta setters become pub(crate)
• Module level comment using //! instead of ///

The change of visibility on the TransactionMeta setters helps us, because if we forget to set a field in either try_from of the Static or Dynamic RuntimeTransaction the compiler will mark these as unused/dead code and warn us the field isn't ever being set.

Fixes #

[codecov]

Codecov Report

Merging #34112 (7d1f761) into master (60fdd85) will increase coverage by 0.0%.
Report is 1 commits behind head on master.
The diff coverage is 0.0%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #34112   +/-   ##
=======================================
  Coverage    81.9%    81.9%           
=======================================
  Files         816      816           
  Lines      219751   219751           
=======================================
+ Hits       180027   180034    +7     
+ Misses      39724    39717    -7     

[tao-stones]

lgtm