Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

camel case prerepairSanityCheck #219

Closed
rootulp opened this issue Jul 5, 2023 · 0 comments · Fixed by #225
Closed

camel case prerepairSanityCheck #219

rootulp opened this issue Jul 5, 2023 · 0 comments · Fixed by #225
Assignees
@rootulp
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@rootulp
Copy link
Collaborator

rootulp commented Jul 5, 2023

rsmt2d/extendeddatacrossword.go

Line 338 in 47826b0

func (eds *ExtendedDataSquare) prerepairSanityCheck(

Since prerepair isn't a word, proposal to rename this function to preRepairSanityCheck so that it's a little easier to read
@rootulp rootulp added enhancement New feature or request good first issue Good for newcomers labels Jul 5, 2023
@rootulp rootulp mentioned this issue Jul 5, 2023
Merged
@rootulp rootulp self-assigned this Jul 5, 2023
rootulp added a commit that referenced this issue Jul 6, 2023
@rootulp
chore: camel case preRepairSanityCheck (#225)
6042512 
Closes #219
0xchainlover pushed a commit to celestia-org/rsmt2d that referenced this issue Aug 1, 2024
@rootulp
chore: camel case preRepairSanityCheck (#225)
e208031 
Closes celestiaorg/rsmt2d#219
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rootulp rootulp

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: camel case preRepairSanityCheck rootulp/rsmt2d
1 participant
@rootulp