Skip to content

Conversation

hdiniz
Copy link
Contributor

@hdiniz hdiniz commented May 28, 2020

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)
  • What kind of change does this PR introduce?
    Bug fix. Return non-zero exit code if library install fails due to dependency resolution.
    Fixes arduino-cli lib install has exit status 0 when library is not found #712

  • What is the current behavior?
    Failure to resolve library dependency is logged, but does not exit with non-zero status code to indicate the error.

  • What is the new behavior?
    If it is not possible to resolve the dependency of a library, the error is logged and the process exits with an error code.

  • Does this PR introduce a breaking change?
    The lib install command will now exit with a non-zero code if a library dependency cannot be resolved.

  • Other information:


See how to contribute

@hdiniz hdiniz changed the title Fixes #712 lib install command will exit with non-zero status code if library dependency could not be resolved May 28, 2020
@cmaglie cmaglie merged commit e4d4651 into arduino:master May 28, 2020
@cmaglie cmaglie added this to the 0.11.0 milestone May 28, 2020
@hdiniz hdiniz deleted the hdiniz/lib-install-exit-code branch May 28, 2020 12:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

arduino-cli lib install has exit status 0 when library is not found
2 participants