fabric-protos icon indicating copy to clipboard operation
fabric-protos copied to clipboard

[WIP] Generate C# bindings

Open jt-nti opened this issue 1 year ago • 8 comments

Just experimenting!

jt-nti avatar Jun 04 '24 13:06 jt-nti

@jt-nti any updates?

Varorbc avatar Jun 20 '24 13:06 Varorbc

@Varorbc unfortunately I'm recovering from Covid at the moment so it will probably take me a bit to get back to this.

Hyperledger has reserved the prefix on NuGet, which is good, but it would help if anyone familiar with publishing packages there could suggest what a "good" C# protobuf package would look like.

jt-nti avatar Jun 21 '24 15:06 jt-nti

@jt-nti I'm very sorry to bother you, and I wish you a speedy recovery.

Perhaps you can send a preview package first, and I will give you some suggestions.

Varorbc avatar Jun 21 '24 21:06 Varorbc

@jt-nti Hello, sorry to bother you again.Any Update?

Varorbc avatar Jul 26 '24 23:07 Varorbc

@Varorbc hi, sorry I haven't had a chance to get back to it so far. I'll try and post something on discord to see if anyone with C#/nuget experience can suggest what needs doing next.

jt-nti avatar Jul 31 '24 10:07 jt-nti

@jt-nti I want to know what is hindering this matter?

Varorbc avatar Jul 31 '24 11:07 Varorbc

@Varorbc it was just a bit of a proof of concept- I think it would be a nice addition but I'm not a C# developer so I have no idea what a good example of this kind of thing should look like. It would also need to be approved and merged by one of the maintainers. The good news is that the Hyperledger prefix has now been reserved in nuget so with a bit more help with credentials etc. it should be possible to publish a package.

jt-nti avatar Aug 01 '24 08:08 jt-nti

@Varorbc getting #228 and #229 merged in the mean time would be good though if possible. It think there was just the C# naming conventions to sort out for SmartBft etc. https://learn.microsoft.com/en-us/dotnet/standard/design-guidelines/capitalization-conventions?redirectedfrom=MSDN

jt-nti avatar Aug 01 '24 08:08 jt-nti