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

Add serial number of connected board to board details command #1216

Merged
merged 2 commits into from
Mar 10, 2021

Conversation

silvanocerza
Copy link
Contributor

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?

Enhances an existing command with more information.

  • What is the current behavior?

board details <fqbn> doesn't return the serial number of a connected board with the specified fqbn.

  • What is the new behavior?

Calling board details <fqbn> when a board with that fqbn is connected that board serial number is returned in the json output and in the gRPC interface function.

Nope.

  • Other information:

None.


See how to contribute

@silvanocerza silvanocerza requested a review from a team March 9, 2021 16:59
@silvanocerza silvanocerza self-assigned this Mar 9, 2021
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.

This is specific to the port, not to the board definition. Boards with the same FQBN can have different SN. So this information should be accessed via arduino-cli board list --format json not board details.

@silvanocerza
Copy link
Contributor Author

@per1234 updated, should be good now.

Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

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

LGTM

@silvanocerza silvanocerza force-pushed the scerza/board-details-serial-number branch from e34031c to 1ff36a6 Compare March 10, 2021 10:58
@silvanocerza
Copy link
Contributor Author

silvanocerza commented Mar 10, 2021

Rebased to fix conflicts.

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 Silvano!

@silvanocerza silvanocerza merged commit 436277f into master Mar 10, 2021
@silvanocerza silvanocerza deleted the scerza/board-details-serial-number branch March 10, 2021 11:56
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.

3 participants