Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

doc: auto gen api #33

Merged
merged 2 commits into from
Oct 29, 2019
Merged

doc: auto gen api #33

merged 2 commits into from
Oct 29, 2019

Conversation

homura
Copy link
Contributor

@homura homura commented Oct 26, 2019

What type of PR is this?
doc
What this PR does / why we need it:
Supports auto generating docs
Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

yejiayu
yejiayu previously approved these changes Oct 28, 2019
pub chain_id: Hash,
#[graphql(description = "Known as the block height like other blockchain")]
pub epoch_id: Uint64,
#[graphql(description = "The merkle root of the previous epoch")]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
#[graphql(description = "The merkle root of the previous epoch")]
#[graphql(description = "Previous epoch hash")]

This is a hash, not a mercke root.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I have make the description more clearly now

@homura homura merged commit ae1a197 into nervosnetwork:master Oct 29, 2019
yejiayu pushed a commit to yejiayu/muta that referenced this pull request Oct 31, 2019
Crate creep Context `with_value()` return value must be used.
yejiayu pushed a commit to yejiayu/muta that referenced this pull request Oct 31, 2019
…ork#33)

* doc: auto gen api

* doc: make the field description more clear
@homura homura deleted the doc-gqlapi branch February 1, 2020 05:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants