Skip to content

GitLab

A
arduino-cli
Switch branch/tag
  1. 01 Feb, 2022 3 commits
  3. 31 Jan, 2022 1 commit
  5. 28 Jan, 2022 5 commits
  7. 26 Jan, 2022 1 commit
  9. 25 Jan, 2022 2 commits
  11. 24 Jan, 2022 3 commits
  13. 21 Jan, 2022 1 commit
    • Cristian Maglie's avatar
      [breaking] legacy: refactoring of the old `i18n.Logger` (part 2) (#1625) · 530e671e
      Cristian Maglie authored 
      * legacy: refactored ErrorfWithLogger function

Now is no longer needed, this function was basically made for
arduino-builder to allow transfering the error to the IDE, BTW the
correct fix must be made inside arduino-builder (to actually let him
push the error in the logger).

* Renamed legacy ctx fields ExecStdout/ExecStderr to Stdout/Stderr

* Removed empty Lint() function

* Removed dependency on i18n.Logger

* Removed no more used i18n.Logger \o/ \o/

* Simplified i18n.Init function

* legacy: builder default output on os.Stdout/os.Stderr

* legacy: updated integration tests for slightly different output of builder

* Use positional parameter for most translated string

This will make easier to transfer translations from the Arduino IDE 1.8

* Added note to UPGRADING.md

* Updated UPGRADING.md

* Removed extra blank line in library detection recap

* Update docs/UPGRADING.md
Co-authored-by: default avatarper1234 <accounts@perglass.com>
      530e671e
  15. 17 Jan, 2022 1 commit
  17. 11 Jan, 2022 1 commit
    • Cristian Maglie's avatar
      [breaking] legacy: refactoring of the old `i18n.Logger` (#1621) · edc63f83
      Cristian Maglie authored 
      * LoggerToCustomStreams must have pointer receiver

Becuase it has a mutex field that otherwise is copied.

* Removed barely used legacy i18n.Logger.UnformattedFprintln function

* Removed barely used legacy i18n.Logger.UnformattedWrite function

* Removed unused AccumulatorLogger

* Added 'percent' to TaskProgress gRPC message

* Added gRPC placeholders to report compile progress

* legacy: builder task progress is now transferred via TaskProgress callback

* Removed unused Logger.Flush interface method

* Removed Logger.Name method (use type-assertions instead)

* Added note on breaking API change
      edc63f83
  19. 03 Jan, 2022 1 commit
  21. 29 Dec, 2021 1 commit
  23. 27 Dec, 2021 1 commit
    • Umberto Baldi's avatar
      [breaking] Optimize `core` operations, improving on the user input (#1574) · e63c798c
      Umberto Baldi authored 
      * [breaking] remove `parseArch` var since it is always true

* [breaking] make packages and platform case insensitive
using the `core.GetPlatform()` approach

* enhance comments and do not optimize if results are != 1

* add logging

* add simple test, install, uninstall etc are already covered since they use the same piece of logic (`ParseReference()`)

* Apply suggestions from code review
Co-authored-by: default avatarper1234 <accounts@perglass.com>

* add new error to handle multiple platform found, return res if the string the user is trying to operate matches perfectly one of the available platforms, optimize the code

* enhance comment describing what the function does

* add test to verify that an operation on two fake cores is not possible

* skip test failing on macOS and on win and optimize the test
Co-authored-by: default avatarper1234 <accounts@perglass.com>
      e63c798c
  25. 23 Dec, 2021 1 commit
  27. 20 Dec, 2021 3 commits
    • per1234's avatar
      Fix install script's check for previous installation (#1603) · 86386576
      per1234 authored 
      The installation script checks for an existing installation in the PATH in order to provide appropriate advice to the
user about adding the installation to their their PATH environment variable.

This check is done using `command -v`. It turns out that the exit status is shell dependent in the event the command is
not found, so that it might be either 1 or 127 depending on the user's system. The script previously assumed that the
exit status would be 1 when the command was not found in PATH, which resulted in spurious advice under these conditions:

```
An existing arduino-cli was found at . Please prepend "/home/foo/arduino-cli/bin" to your $PATH or remove the existing one.
```

It seems safest to fix this by inverting the logic so that the advice about an existing installation in PATH is only
printed when one was found.
      86386576
    • Paolo Calao's avatar
      Add YAML output format (#1600) · 5dd14c4c
      Paolo Calao authored
      5dd14c4c
    • per1234's avatar
      [skip changelog] Use major version ref of `carlosperate/download-file-action` (#1602) · 8dcea2d6
      per1234 authored 
      The `carlosperate/download-file-action` action is used in the GitHub Actions workflows as a convenient way to download
external resources.

A major version ref has been added to that repository. It will always point to the latest release of the "1" major
version series. This means it is no longer necessary to do a full pin of the action version in use as before.

Use of the major version ref will cause the workflow to use a stable version of the action, while also benefiting from
ongoing development to the action up until such time as a new major release of an action is made. At that time we would
need to evaluate whether any changes to the workflow are required by the breaking change that triggered the major
release before manually updating the major ref (e.g., uses: `carlosperate/download-file-action@v2`). I think this
approach strikes the right balance between stability and maintainability for these workflows.
      8dcea2d6
  29. 07 Dec, 2021 4 commits
  31. 03 Dec, 2021 2 commits
  33. 02 Dec, 2021 2 commits
    • Silvano Cerza's avatar
      Update python version in integration tests (#1579) · 822134d9
      Silvano Cerza authored
      822134d9
    • Silvano Cerza's avatar
      Enhance i18n:check task (#1578) · 8f0e8aec
      Silvano Cerza authored 
      i18n:check task used to verify only if the source en.po file was up to
date. The new process instead verifies that all translations are up to
date by downloading them directly from Transifex.

The previous process was causing some issues since after uploading the
en.po file Transifex would change its formatting, meaning when calling
i18n:pull we would download a file containing the same information but
formatted in a different way and that made the i18n:check command fail.

By making the i18n:check task call i18n:pull instead of i18n:update we
don't have issues with the formatting anymore.

This change also covers a corner-case that would occur when the
en.po file wasn't updated since a long time, the i18n:check wouldn't
fail because the en.po was the same as before but that wouldn't mean
that the translations were up to date with the latest from Transifex.
      8f0e8aec
  35. 30 Nov, 2021 1 commit
  37. 26 Nov, 2021 1 commit
  39. 25 Nov, 2021 2 commits
  41. 23 Nov, 2021 2 commits
  43. 22 Nov, 2021 1 commit