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

Update helper functions documentation #557

Merged
merged 5 commits into from
Apr 5, 2019

Conversation

akhila-raju
Copy link
Contributor

@akhila-raju akhila-raju commented Apr 5, 2019

PR Description

Updates helper functions documentation

@akhila-raju akhila-raju added the doc-change-required Indicates an issue or PR that requires doc to be updated label Apr 5, 2019
@akhila-raju akhila-raju added this to the Beacon Chain v0.5.1 Spec milestone Apr 5, 2019
@akhila-raju akhila-raju self-assigned this Apr 5, 2019
@akhila-raju akhila-raju changed the title [WIP] Update helpers Update helper documentation Apr 5, 2019
@akhila-raju akhila-raju changed the title Update helper documentation Update helper functions documentation Apr 5, 2019
Copy link
Contributor

@schroedingerscode schroedingerscode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a naming preference...your call on that one.

@schroedingerscode schroedingerscode dismissed their stale review April 5, 2019 16:12

Misread a var when I was reviewing.

@akhila-raju akhila-raju merged commit 2c5719f into Consensys:v0.5.1-integration Apr 5, 2019
@akhila-raju akhila-raju deleted the updatehelpers branch April 5, 2019 19:08
akhila-raju pushed a commit that referenced this pull request Apr 5, 2019
cemozerr pushed a commit that referenced this pull request Apr 5, 2019
@alexandratran alexandratran removed the doc-change-required Indicates an issue or PR that requires doc to be updated label Mar 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants