STR-211: CI build checks for the prover guest code

[evgenyzdanovich]

Description

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactor
• New or updated tests
• Dependency Update

Notes to Reviewers

Checklist

• I have performed a self-review of my code.
• I have commented my code where necessary.
• I have updated the documentation if needed.
• My changes do not introduce new warnings.
• I have added tests that prove my changes are effective or that my feature works.
• New and existing tests pass with my changes.

Related Issues

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.55%. Comparing base (f07bab7) to head (ec76424).
Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main     #512      +/-   ##
==========================================
- Coverage   56.58%   56.55%   -0.03%     
==========================================
  Files         275      275              
  Lines       29192    29192              
==========================================
- Hits        16517    16510       -7     
- Misses      12675    12682       +7     

see 5 files with indirect coverage changes

[MdTeach]

Should we also add build check for the prover-client binary here ?
I think in normal CI build it will be also skipped

strata/bin/prover-client/Cargo.toml

Lines 6 to 9 in 08ea646

	[[bin]]
	name = "strata-prover-client"
	path = "src/main.rs"
	required-features = ["prover"]

[evgenyzdanovich]

Should we also add build check for the prover-client binary here ? I think in normal CI build it will be also skipped

strata/bin/prover-client/Cargo.toml

Lines 6 to 9 in 08ea646

	[[bin]]
	name = "strata-prover-client"
	path = "src/main.rs"
	required-features = ["prover"]

@MdTeach Thanks a lot!!! You are completely right. I missed one directory, didn't realize that there's no cargo.toml under provers/ and it was building the whole thing (including reth and all other binaries).

Trying to restrict it.

[delbonis]

Why were all those files deleted?

[evgenyzdanovich]

@delbonis to not bloat the actions tab on the repository, because I experiment a lot and run CI on this branch frequently.
I'll add it back definitely once the PR is ready for the review.