Skip to content

Commit 21e2f2f

Browse files
committed
Merge #11135: Update developer notes with RPC response guidelines
62ecce7 [doc] Add RPC response notes (João Barbosa) Pull request description: Tree-SHA512: 795969813eefab47ac03050bd70762fc36d86edb00bdfe1ba76837a810370d61dc88749c51153bd3949eb3885f084b4ba8736b4bbf90dfef65e3fe1939304bd8
2 parents 5368614 + 62ecce7 commit 21e2f2f

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

doc/developer-notes.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -619,3 +619,8 @@ A few guidelines for introducing and reviewing new RPC interfaces:
619619
- *Rationale*: as well as complicating the implementation and interfering
620620
with the introduction of multi-wallet, wallet and non-wallet code should be
621621
separated to avoid introducing circular dependencies between code units.
622+
623+
- Try to make the RPC response a JSON object.
624+
625+
- *Rationale*: If a RPC response is not a JSON object then it is harder to avoid API breakage if
626+
new data in the response is needed.

0 commit comments

Comments
 (0)