Open xJonathanLEI opened 5 months ago
Fixes a long-standing issue where the absence of required is ambiguous: it could either mean all fields are optional, or that none of them are:
required
declared_classes
TXN_RECEIPT_WITH_BLOCK_INFO
This makes code generation really difficult as codegen tools have to make external assumptions on what the intended behavior is.
With this PR, now all properties are always paired with a valid required field.
properties
Fixes a long-standing issue where the absence of
requiredis ambiguous: it could either mean all fields are optional, or that none of them are:requiredin thedeclared_classesfield names they are always present.requiredinTXN_RECEIPT_WITH_BLOCK_INFOmeans they're all optional.This makes code generation really difficult as codegen tools have to make external assumptions on what the intended behavior is.
With this PR, now all
propertiesare always paired with a validrequiredfield.