bendk

Results 297 comments of bendk

> Created the initial types and methods for the API. I think all of these should be considered works-in-progress. It seems likely that some or all of them will change...

I'm pushing an updated version of this that: - Fixes the merge conflicts - Removes the commit that was testing if we can use proc-macros. I'd rather switch over fully...

I think this is getting pretty close to ready. @jeddai Do the changes to [run_python_tests.sh](https://github.com/mozilla/application-services/pull/5592/files#diff-caeeb7f604b92f1acb365eb94492a921c3cc2eab0348b6dca677d8ef9b21f95b) look good to you? I'm actually a bit jealous of the cirrus situation, you're the...

@mhammond what do you think about merging this one to main, maybe after 117 starts?

> Maybe we can try to land https://github.com/mozilla/application-services/pull/5704 and use that as a confidence builder as well? +1 to this, I think that will go a long way.

Made https://bugzilla.mozilla.org/show_bug.cgi?id=1867813 for upgrading the mac worker.

Coming in late to this one, and I'm not sure I understand it all, but I wonder if it would be possible to rewrite each method by convert the `cfg_attr`...

Maybe we could simplify the parsing, if we told users they had to use our syntax for conditionals: ``` #[uniffi::export)] impl SomeObject { #[cfg_attr(wasm, wasm_bindgen(constructor))] #[uniffi::constructor(name="new", cfg_if=all(not(wasm), target_os = "android"))]...

I think these approaches should work: - The `ExternalInterface` in a UDL file you laid out - Importing the type (`use RETDecimal from radix_engine_toolkit_uniffi`) - Defining the `Decimal(NativeDecimal)` struct with...

> Instead of having to put an entry for each RET type (and function?) in Sargon's UDL, is it somehow possible to in Sargon say "I wanna RE-EXPORT the whole...