You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Nov 14, 2024. It is now read-only.
You need to map Apps Script's "Integer" datatype to ts' "number" primitive. It looks like you are already correctly handling the casing of boolean and string (upper case in gas docs, lower case in ts types).
You might also take a look to see if you want to emit "any" instead of "Object". There may be cases where an Apps Script developer is used to calling prototype methods on the return, which in turn the Apps Script runtime is pretty good at handling. The ts compiler would fail on Object, but would allow on 'any'.
The text was updated successfully, but these errors were encountered:
For the Object case, it seems reasonable to replace or assign Object to 'any'. If you don't mind providing some test cases, it would be easier to implement this. Thanks!
Since you are just mapping Apps Script's Integer to ts' number type, it probably is OK doing that explicitly, but you could wait on for more votes than just my 1.
I also saw a few instances of String[] that need to be lowercased. For example, in the spreadsheet class DataValidationBuilder#requireValueInList.
You need to map Apps Script's "Integer" datatype to ts' "number" primitive. It looks like you are already correctly handling the casing of boolean and string (upper case in gas docs, lower case in ts types).
You might also take a look to see if you want to emit "any" instead of "Object". There may be cases where an Apps Script developer is used to calling prototype methods on the return, which in turn the Apps Script runtime is pretty good at handling. The ts compiler would fail on Object, but would allow on 'any'.
The text was updated successfully, but these errors were encountered: