typed chain ID

[tzemanovic]

Describe your changes

Switched to use typed ChainId instead of a string in all fn get_chain_id.

Depends-On: #3718

diff: https://github.com/anoma/namada/pull/3733/files/c6209226992e444d46abae9173baeb09b76f3fcc..d904873f63b0def8451c0399441375f3c8db0c0c

Checklist before merging

• If this PR has some consensus breaking changes, I added the corresponding breaking:: labels
  • This will require 2 reviewers to approve the changes

[mergify]

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

⛓️ Depends-On Requirements

Requirement based on the presence of Depends-On in the body of the pull request

• depends-on=anoma/namada#3718 [⛓️ mv gov sub-module from shell into gov crate #3718]

[codecov]

Codecov Report

Attention: Patch coverage is 82.22222% with 112 lines in your changes missing coverage. Please review.

Project coverage is 72.47%. Comparing base (ef00f27) to head (8271805).
Report is 27 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/governance/src/finalize_block.rs	81.86%	78 Missing ⚠️
crates/node/src/shell/finalize_block.rs	72.97%	20 Missing ⚠️
crates/node/src/protocol.rs	0.00%	3 Missing ⚠️
crates/ibc/src/vp/context.rs	0.00%	2 Missing ⚠️
crates/storage/src/lib.rs	0.00%	2 Missing ⚠️
crates/trans_token/src/lib.rs	95.45%	2 Missing ⚠️
crates/vp/src/native_vp.rs	33.33%	2 Missing ⚠️
crates/vp_prelude/src/lib.rs	71.42%	2 Missing ⚠️
crates/ibc/src/actions.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3733      +/-   ##
==========================================
+ Coverage   72.41%   72.47%   +0.05%     
==========================================
  Files         335      334       -1     
  Lines      104005   104129     +124     
==========================================
+ Hits        75320    75468     +148     
+ Misses      28685    28661      -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mergify]

Hey @tzemanovic, your pull request has been dequeued due to the following reason: CHECKS_FAILED.
Sorry about that, but you can requeue the PR by using @mergifyio requeue if you think this was a mistake.

[tzemanovic]

@Mergifyio requeue

[mergify]

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically