diff options
| author | Pavlo Strokov <pstrokov@gitlab.com> | 2022-08-10 12:24:32 +0300 |
|---|---|---|
| committer | Pavlo Strokov <pstrokov@gitlab.com> | 2022-08-22 17:55:03 +0300 |
| commit | 1a8791a4d7cb198218b95de1de74c96d37c0b80a (patch) | |
| tree | 5de2e626fa098b2807280b477c7fd7b0d3d0a57b /ruby/proto | |
| parent | 46d8149fe7233a1d0d7518f64ace97bccba3ad1b (diff) | |
proto: Extend FindLicenseResponse with additional fields
Gitaly uses the Licensee gem to find the license used in a
repository and return the corresponding license identifier
to GitLab. GitLab uses the same Licensee gem to get additional
info about the license using the identifier to show it on
the UI. Go and Ruby implementations of the FindLicense RPC
rely on different algorithms for license detection and the
set of licenses that could be detected are different as well,
that is why they are not fully compatible. Let's extend
FindLicenseResponse with additional fields, so GitLab
doesn't need to get that info from the Licensee gem anymore.
It solves the problem of the compatibility of the two
implementations.
Part of: https://gitlab.com/gitlab-org/gitaly/-/issues/3782
Diffstat (limited to 'ruby/proto')
| -rw-r--r-- | ruby/proto/gitaly/repository_pb.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/ruby/proto/gitaly/repository_pb.rb b/ruby/proto/gitaly/repository_pb.rb index 64c5e03e9..81df3add0 100644 --- a/ruby/proto/gitaly/repository_pb.rb +++ b/ruby/proto/gitaly/repository_pb.rb @@ -202,6 +202,10 @@ Google::Protobuf::DescriptorPool.generated_pool.build do end add_message "gitaly.FindLicenseResponse" do optional :license_short_name, :string, 1 + optional :license_name, :string, 2 + optional :license_url, :string, 3 + optional :license_path, :string, 4 + optional :license_nickname, :string, 5 end add_message "gitaly.GetInfoAttributesRequest" do optional :repository, :message, 1, "gitaly.Repository" |