1. 22 Mar, 2022 1 commit
    • per1234's avatar
      [skip changelog] Disable internal anchor checks by link checker tool (#1692) · f470f407
      per1234 authored
      Validation of links to internal anchors was recently added to the markdown-link-check tool used to check for broken
      links in this project's documentation.
      
      Although a much needed feature, unfortunately it does not currently support anchors created by HTML anchor tags.
      
      For example, this is valid and common Markdown:
      
      [click here](#some-anchor)
      <a name="some-anchor"></a>
      
      but will fail the check:
      
      ERROR: 1 dead links found!
      [x] #some-anchor -> Status: 404
      
      This type of link markup is used by the protoc-gen-doc tool that generates the gRPC interface documentation, which
      causes a spurious failure of the link check.
      
      The solution is to configure markdown-link-check to skip these links (which was the behavior anyway with versions 3.8.7
      and older). That will be reverted whenever the tool is able to correctly validate internal anchor links.
      f470f407
  2. 14 Mar, 2022 1 commit
  3. 10 Mar, 2022 2 commits
  4. 04 Mar, 2022 1 commit
    • Jesse Vincent's avatar
      [skip changelog] Make use of 'avr' as example platform more consistent (#1680) · 8b53b850
      Jesse Vincent authored
      * Make use of 'avr' as example platform more consistent
      
      It's fairly easy to miss the one line early in the doc that says this text uses 'avr' as an example, leading a reader to possibly believe the documentation hasn't been updated since 'avr' was the only platform choice. Within the doc, the text was a little inconsistent as to how it referred to the avr platform and platform specific tools. This change tries to standardize how we refer to the example platform.
      
      * Fix a couple lint errors.
      
      * Fix prettier lint errors.
      
      * Fix an additional prettier lint error
      
      * Update docs/sketch-build-process.md
      Co-authored-by: default avatarper1234 <accounts@perglass.com>
      Co-authored-by: default avatarper1234 <accounts@perglass.com>
      8b53b850
  5. 03 Mar, 2022 1 commit
  6. 02 Mar, 2022 1 commit
  7. 01 Mar, 2022 1 commit
  8. 23 Feb, 2022 3 commits
  9. 18 Feb, 2022 1 commit
  10. 17 Feb, 2022 2 commits
  11. 04 Feb, 2022 2 commits
  12. 03 Feb, 2022 1 commit
  13. 01 Feb, 2022 3 commits
  14. 31 Jan, 2022 1 commit
  15. 28 Jan, 2022 5 commits
  16. 26 Jan, 2022 1 commit
  17. 25 Jan, 2022 2 commits
  18. 24 Jan, 2022 3 commits
  19. 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
  20. 17 Jan, 2022 1 commit
  21. 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
  22. 03 Jan, 2022 1 commit
  23. 29 Dec, 2021 1 commit
  24. 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
  26. 20 Dec, 2021 1 commit
    • 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