search

icerpc / icerpc-csharp

A C# RPC framework built for QUIC, with bidirectional streaming, first-class async/await, and Protobuf support.
https://docs.icerpc.dev
Apache License 2.0
107 stars 13 forks source link

Tools layout #3709

Closed pepone closed 1 year ago

pepone commented 1 year ago

The current layout for tools is not great

tools/IceRpc.Slice.Tools
tools/protobuf/Common
tools/protobuf/IceRpc.Protobuf.Tools
tools/protobuf/IceRpc.ProtocGen
tools/slicec-cs

I think it would make more sense to move the Slice-related tools into a new slice subdirectory.

tools/protobuf/Common
tools/protobuf/IceRpc.Protobuf.Tools
tools/protobuf/IceRpc.ProtocGen
tools/slice/slicec-cs
tools/slice/IceRpc.Slice.Tools
InsertCreativityHere commented 1 year ago

I agree, that seems more logical now that it's split between protobuf and slice.

pepone commented 1 year ago

@bernardnormier suggested moving everything to the tools top-level:

tools/Common
tools/IceRpc.Protobuf.Tools
tools/IceRpc.ProtocGen
tools/slicec-cs
tools/IceRpc.Slice.Tools

And we will have a single IceRpc.Tools.sln for the C# projects