Skip to content
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

[WIP] Generate C# bindings #230

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

jt-nti
Copy link
Member

@jt-nti jt-nti commented Jun 4, 2024

Just experimenting!

@jt-nti jt-nti force-pushed the csharp-bindings branch 3 times, most recently from 48363b7 to 7edda34 Compare June 6, 2024 11:59
Just experimenting!

Signed-off-by: James Taylor <[email protected]>
@Varorbc
Copy link

Varorbc commented Jun 20, 2024

@jt-nti any updates?

@jt-nti
Copy link
Member Author

jt-nti commented Jun 21, 2024

@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.

@Varorbc
Copy link

Varorbc commented Jun 21, 2024

@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
Copy link

Varorbc commented Jul 26, 2024

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

@jt-nti
Copy link
Member Author

jt-nti commented Jul 31, 2024

@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.

@Varorbc
Copy link

Varorbc commented Jul 31, 2024

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

@jt-nti
Copy link
Member Author

jt-nti commented Aug 1, 2024

@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
Copy link
Member Author

jt-nti commented Aug 1, 2024

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants