-
Notifications
You must be signed in to change notification settings - Fork 59
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
Generate compiler api in TypeScript using autorest #1456
Conversation
validateBytecode, | ||
...otherOptions | ||
} = {}) { | ||
if (!_aci && source) { | ||
_aci = await this.compilerApi.generateACI({ code: source, options: { filesystem } }) | ||
_aci = await this.compilerApi.generateACI({ code: source, options: { fileSystem } }) |
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.
for some reason sdk and aesophia_http names this option differently
6504ec6
to
b824bb6
Compare
b824bb6
to
7a61a2f
Compare
Apart from the need of rebasing, I think it looks great! I could also suggest updating the testing docs so contributors know that they need to run "test": "checkApi && nyc mocha './test/unit/' './test/integration/'", Finally, since |
7a61a2f
to
a9cfb13
Compare
BREAKING CHANGE: `filesystem` option renamed to `fileSystem` BREAKING CHANGE: methods of `compilerApi` requires `options` object according to their specification BREAKING CHANGE: methods of `compilerApi` returns and accepts keys named in camelCase instead of snake_case
BREAKING CHANGE: `ContractCompilerHttp` creates and changes compiler URL in sync Don't tread `ContractCompilerHttp(...)` and `compiler.setCompilerUrl(...)` as a `Promise` anymore. BREAKING CHANGE: `ContractCompilerHttp` doesn't check version on creating and switching the url This would be checked before the first request.
a9cfb13
to
b0c33b3
Compare
fixes partially #1216
found issues: aeternity/aesophia_http#87 aeternity/aesophia_http#88 aeternity/aesophia_http#90 aeternity/aesophia_http#91