-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ISSUE#405 dev programs rework #409
Conversation
…the type interface of ProgramInstance for registration
All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Requested changes that I think are important are marked with 🔥
- 🔥 CHANGELOG needed
- 🔥 put some check in to catch inevitable accidental misuse of new `get
- 🔥 fix failing test
What test is failing? |
The one in this comment - #409 (comment) |
ahh missed that in the comment. |
Co-authored-by: mix irving <mix@protozoa.nz>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My only 🔥 is I'm pretty concerned about giving up on Schema Parsing.
I would probably throw an error, and if you want, attach the data did some out ok to that error?
* @property {ArrayBuffer} bytecode - The bytecode of the program. | ||
* @property {unknown} [interfaceDescription] - Optional. The configuration interface of the program. | ||
* @property {string} deployer - The address of the deployer of the program. | ||
* @property {number} refCounter - The reference count for the program. | ||
*/ | ||
|
||
// interfaceDescription needs better design and another type other than 'unknown' | ||
export interface ProgramInfo { | ||
export interface ProgramInterface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
before release, lets see if we can get this exported for use in CLI / other apps
Co-authored-by: mix irving <mix@protozoa.nz>
Co-authored-by: mix irving <mix@protozoa.nz>
…o frankie/i#405programs
Related Issue(s)
closes #405
Testing
Checklist