Skip to content

[breaking] Removed real_name field from gRPC Library message #1890

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

Merged
merged 2 commits into from
Oct 4, 2022

Conversation

cmaglie
Copy link
Member

@cmaglie cmaglie commented Sep 23, 2022

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • UPGRADING.md has been updated with a migration guide (for breaking changes)
  • What kind of change does this PR introduce?
    Removed the real_name field from gRPC API commands.v1.Library
  • What is the current behavior?
    Library.real_name and Library.name have the same values and real_name is deprecated.
  • What is the new behavior?
    real_name has been removed
  • Does this PR introduce a breaking change, and is
    titled accordingly?

    Yes, as described above

@cmaglie cmaglie self-assigned this Sep 23, 2022
@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Base: 36.58% // Head: 36.74% // Increases project coverage by +0.16% 🎉

Coverage data is based on head (6c3e203) compared to base (5730e2e).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1890      +/-   ##
==========================================
+ Coverage   36.58%   36.74%   +0.16%     
==========================================
  Files         231      231              
  Lines       19696    19720      +24     
==========================================
+ Hits         7206     7247      +41     
+ Misses      11659    11642      -17     
  Partials      831      831              
Flag Coverage Δ
unit 36.74% <0.00%> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
arduino/libraries/libraries.go 43.85% <ø> (+0.38%) ⬆️
cli/lib/list.go 0.00% <0.00%> (ø)
arduino/cores/packagemanager/loader.go 72.21% <0.00%> (-0.10%) ⬇️
internal/integrationtest/arduino-cli.go 85.66% <0.00%> (+0.31%) ⬆️
arduino/cores/cores.go 62.17% <0.00%> (+0.59%) ⬆️
arduino/cores/packagemanager/package_manager.go 66.58% <0.00%> (+0.75%) ⬆️
arduino/cores/packagemanager/download.go 27.71% <0.00%> (+4.81%) ⬆️
commands/lib/search.go 92.50% <0.00%> (+12.83%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@cmaglie cmaglie force-pushed the grpc_cleanup branch 2 times, most recently from 3b474e0 to 8e133de Compare September 23, 2022 16:00
@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself topic: CLI Related to the command line interface topic: gRPC Related to the gRPC interface labels Oct 2, 2022
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Cristian!

@cmaglie cmaglie merged commit bcb69d9 into arduino:master Oct 4, 2022
@cmaglie cmaglie deleted the grpc_cleanup branch October 4, 2022 15:00
kittaakos pushed a commit to arduino/arduino-ide that referenced this pull request Oct 5, 2022
It has been removed from the gRPC API: arduino/arduino-cli#1890

This PR switches from `real_name` to `name` in the UI, as the `name` is
the canonical form provided by the CLI.

Closes #1525

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit to arduino/arduino-ide that referenced this pull request Oct 5, 2022
It has been removed from the gRPC API: arduino/arduino-cli#1890

This PR switches from `real_name` to `name` in the UI, as the `name` is
the canonical form provided by the CLI.

Closes #1525

Signed-off-by: Akos Kitta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: CLI Related to the command line interface topic: code Related to content of the project itself topic: gRPC Related to the gRPC interface type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants