Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Updating documentation for RPCs #5392

Merged
merged 6 commits into from
Apr 8, 2017
Merged

Updating documentation for RPCs #5392

merged 6 commits into from
Apr 8, 2017

Conversation

tomusdrw
Copy link
Collaborator

@tomusdrw tomusdrw commented Apr 4, 2017

@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. F5-documentation 📑 Documentation needs fixing, improving or augmenting. labels Apr 4, 2017
Copy link
Contributor

@maciejhirsz maciejhirsz left a comment

Choose a reason for hiding this comment

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

Would be good if we had a systemic way to mark deprecated stuff, but otherwise looks fine.

There are linter errors.

@maciejhirsz maciejhirsz added A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. and removed A0-pleasereview 🤓 Pull request needs code review. labels Apr 4, 2017
@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. and removed A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Apr 4, 2017
@gavofyork
Copy link
Contributor

JS still failing here @maciejhirsz / @tomusdrw

}

return derive.map(item => {
const index = inNumber10(item && item.index ? item.index : item);
Copy link
Contributor

Choose a reason for hiding this comment

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

Linter requires a new line


function inDeriveType (derive) {
return derive && derive.type === 'hard' ? 'hard' : 'soft';
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Would put a new line after here as well

Copy link
Contributor

@maciejhirsz maciejhirsz left a comment

Choose a reason for hiding this comment

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

Just going by the linter error from CI logs.

@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. M6-rpcapi 📣 RPC API. and removed A0-pleasereview 🤓 Pull request needs code review. F5-documentation 📑 Documentation needs fixing, improving or augmenting. labels Apr 8, 2017
@gavofyork gavofyork merged commit 4e49879 into master Apr 8, 2017
@gavofyork gavofyork deleted the minblock-docs branch April 8, 2017 09:28
tomusdrw added a commit that referenced this pull request May 18, 2017
* Removing minBlocks occurrencies

* Docs for new RPCs.

* Fixing linting issues, updating *withToken documentatiojn.

* Adding missing RPCs. Fixing tests.

* Fixing lint issues.
arkpar pushed a commit that referenced this pull request May 18, 2017
* option to disable persistent txqueue (#5544)

* option to disable persistent txqueue

* New option goes with kin

* Remove transaction RPC (#4949)

* Cancel tx JS (#4958)

* Remove transaction RPC

* Bumping multihash and libc

* Updating nanomsg

* bump nanomsg

* cancel tx

* cancel-tx-js

* cancel-tx-js

* cancel-tx-js

* cancel-tx-hs

* cancel-tx-js

* cancel-tx-js

* cancel-tx-js

* small fixes

* edit & time till submit

* edit & time till submit

* updates

* updates

* udpates

* udpates

* grumbles

* step 1

* Wonderful updates

* ready

* small refact

* small refact

* grumbles 1

* ffx2

* good ol' fashioned updates

* latest and greatest

* removeHash

* removeHash

* spec

* fix 1

* fix 1

* fix 2

* fix 2

* ff

* ff

* ff

* updates

* Updating documentation for RPCs (#5392)

* Removing minBlocks occurrencies

* Docs for new RPCs.

* Fixing linting issues, updating *withToken documentatiojn.

* Adding missing RPCs. Fixing tests.

* Fixing lint issues.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M6-rpcapi 📣 RPC API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants