univalue: Return more detailed type check error messages #25629

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2207-json-push-🎬 changing 11 files +54 −50
  1. MarcoFalke commented at 9:14 am on July 18, 2022: member
    Print the current type and the expected type
  2. move-only: Move UniValue::getInt definition to keep class with definitions only
    Can be reviewed with the git options
    
    --color-moved=dimmed-zebra --color-moved-ws=ignore-all-space
    fafab147e7
  3. MarcoFalke added the label RPC/REST/ZMQ on Jul 18, 2022
  4. MarcoFalke removed the label RPC/REST/ZMQ on Jul 18, 2022
  5. MarcoFalke commented at 9:14 am on July 18, 2022: member
    (Somewhat) requested by @furszy in #25551#pullrequestreview-1038990017
  6. DrahtBot added the label RPC/REST/ZMQ on Jul 18, 2022
  7. univalue: Return more detailed type check error messages fae5ce8795
  8. MarcoFalke force-pushed on Jul 18, 2022
  9. aureleoules commented at 1:17 pm on July 18, 2022: member
    ACK fae5ce8795080018875227aee8613677f92e99ce. Good refactor, it’s easier to check types this way 👍.
  10. DrahtBot commented at 8:16 pm on July 18, 2022: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #23319 (rpc: Return fee and prevout (utxos) to getrawtransaction by dougEfresh)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  11. fanquake merged this on Jul 19, 2022
  12. fanquake closed this on Jul 19, 2022

  13. MarcoFalke deleted the branch on Jul 19, 2022
  14. sidhujag referenced this in commit 698475fcfa on Jul 19, 2022
  15. furszy referenced this in commit 8a8583d663 on Jul 29, 2022
  16. furszy referenced this in commit a3e5674df3 on Jul 29, 2022
  17. furszy referenced this in commit c2cc1db1d1 on Jul 29, 2022
  18. furszy referenced this in commit 0e5e63cc31 on Jul 29, 2022
  19. furszy referenced this in commit b629456c59 on Jul 29, 2022
  20. furszy referenced this in commit f6f80d459b on Jul 29, 2022
  21. furszy referenced this in commit 4e108c172a on Jul 29, 2022
  22. furszy referenced this in commit 1fbf52033e on Jul 29, 2022
  23. furszy referenced this in commit 53ce3b886f on Jul 29, 2022
  24. furszy referenced this in commit d48f5d1a65 on Jul 30, 2022
  25. furszy referenced this in commit 133fa8add3 on Jul 31, 2022
  26. furszy referenced this in commit d54f65c5d1 on Jul 31, 2022
  27. furszy referenced this in commit f1ee517adf on Sep 2, 2022
  28. furszy referenced this in commit 5ea10772a1 on Sep 2, 2022
  29. furszy referenced this in commit 549fb177aa on Sep 8, 2022
  30. furszy referenced this in commit d8d9edee45 on Sep 14, 2022
  31. furszy referenced this in commit 8bdd192d4f on Sep 14, 2022
  32. furszy referenced this in commit 5d9ea95428 on Sep 15, 2022
  33. furszy referenced this in commit e68d380797 on Sep 15, 2022
  34. fanquake referenced this in commit b1f44ecdcd on Sep 21, 2022
  35. sidhujag referenced this in commit c366c50c21 on Sep 23, 2022
  36. bitcoin locked this on Jul 19, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-18 18:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me