chore: namespaced padding share isSequenceStart=false

[rootulp]

Closes #1245

• Extract NamespacedPaddedShare to a stand-alone file
• Modify namespaced padded share to set isSequenceStart=false
• Resolve a TODO by moving tests to parse_sparse_shares_test.go

[codecov-commenter]

Codecov Report

Merging #1246 (ef002ec) into main (c74c614) will increase coverage by 0.77%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##             main    #1246      +/-   ##
==========================================
+ Coverage   47.99%   48.76%   +0.77%     
==========================================
  Files          72       75       +3     
  Lines        4084     4339     +255     
==========================================
+ Hits         1960     2116     +156     
- Misses       1952     2043      +91     
- Partials      172      180       +8     

Impacted Files	Coverage Δ
pkg/shares/parse_sparse_shares.go	66.03% <ø> (-2.93%)	⬇️
pkg/shares/namespaced_padding.go	88.23% <88.23%> (ø)
pkg/shares/share_splitting.go	71.60% <100.00%> (ø)
pkg/shares/split_sparse_shares.go	65.16% <100.00%> (-4.28%)	⬇️
pkg/prove/proof.go	75.84% <0.00%> (-10.08%)	⬇️
app/estimate_square_size.go	88.46% <0.00%> (-6.67%)	⬇️
app/write_square.go	90.90% <0.00%> (-2.08%)	⬇️
testutil/testnode/node_interaction_api.go	56.41% <0.00%> (-1.45%)	⬇️
x/blob/types/blob_tx.go	18.57% <0.00%> (-1.13%)	⬇️
x/blob/types/payforblob.go	81.09% <0.00%> (-0.73%)	⬇️
... and 10 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[cmwaters]

Nice work. I just have one question

[MSevey]

I don't think I did.... something weird seems to be happening with the github action maybe.

[cmwaters]

I think that's just how the bot works. Also whenever I change is made to a PR, all prior reviews are dismissed so the request for reviews happens again