Alberto Penas Amor

Results 20 comments of Alberto Penas Amor

Hi! Any updates on this @adamrushy? :) https://github.com/adamrushy/OpenAISwift/pull/33

@adamrushy @MarkHoath Solved conflicts, ready to review and merge! Could you take a look @adamrushy Take in mind that I have added unit tests to the completions endpoint. Now on...

@pvieito thanks for your feedback, applied all your suggestions :)

@adamrushy could you take a look?

PR #783 opened @janndriessen Could you take a look? If the purpose was wrongly understood let me know :)

@yaroslavyaroslav I would like to understand what is your idea about this FIXME. I will be reading the code and figure out what can I purpose as a solution from...

I have checking and `WriteOperation` inherits ReadOperation and has a FIXME `FIXME: Rewrite this to CodableTransaction`. We should apply same/similar strategy

To be honest I don't get the point of moving `ReadOperation` to `CodableTransaction`. This Read/Write is highly coupled with the contract itself not a transaction(If I am wrong let me...

ok, then it should be enough to do points 1 and 2? @yaroslavyaroslav

I understand, I guess you should add your comment to the readme to avoid future issues