Skip to content

Conversation

@dido18
Copy link
Contributor

@dido18 dido18 commented Oct 15, 2025

Remove external dependencies by copying/pasting the i18n and feedback packages here.

Other

  • renamed the module to github.com/arduino/arduino-flasher-cli
  • updated the license header of the packages with the command addlicense -c "ARDUINO SA (http://www.arduino.cc/)" -f ./license_header.tpl $(find . -name "*.go" -type f -print0 | xargs -0)

Other notes

  • The i18n can be removed. Translation can be added when needed
  • revise the usage of the feedack package, remove not used code

@dido18 dido18 requested review from a team and MatteoPologruto October 15, 2025 09:52
@dido18
Copy link
Contributor Author

dido18 commented Oct 15, 2025

Build tested locally with go build -ldflags "-X main.Version=1.0.0" -v -o .

Copy link
Contributor

@MatteoPologruto MatteoPologruto left a comment

Choose a reason for hiding this comment

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

Just some nitpicks

dido18 and others added 3 commits October 15, 2025 12:20
Co-authored-by: MatteoPologruto <109663225+MatteoPologruto@users.noreply.github.com>
@dido18 dido18 merged commit 8f86284 into main Oct 15, 2025
@dido18 dido18 deleted the remove-orch-dependencies branch October 15, 2025 10:46
lucarin91 pushed a commit that referenced this pull request Oct 28, 2025
* feat: refactor import paths and add feedback and i18n packages

* feat: update copyright and licensing information in source files

Co-authored-by: MatteoPologruto <109663225+MatteoPologruto@users.noreply.github.com>

---------

Co-authored-by: MatteoPologruto <109663225+MatteoPologruto@users.noreply.github.com>
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.

2 participants