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.

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

Replaced the conflicting vrf_str variable name with vrf_text thro… #17046

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shubhamvk03
Copy link

Replaced the conflicting vrf_str variable name with vrf_text throughout the code to resolve naming conflicts with the vrf_str() function.

…ughout the code to resolve naming conflicts with the `vrf_str()` function.
@donaldsharp
Copy link
Member

this pr has some major problems
a) it doesn't follow our commit guidelines about how the commit message should be formated
b) the commit has multiple unrelated changes to what is described
c) Looks like a fat finger removal of a non-trivial number of lines occurred.

@shubhamvk03 shubhamvk03 reopened this Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants