Skip to content

Conversation

@lucarin91
Copy link
Contributor

@lucarin91 lucarin91 commented Oct 15, 2025

Motivation

Add a CI workflow for testing and checking the code. Add licenses and the CI to be sure we are license-compliant.

Change description

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

Copy link
Contributor

@dido18 dido18 left a comment

Choose a reason for hiding this comment

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

LGTM

@lucarin91 lucarin91 requested a review from per1234 October 15, 2025 16:28
@lucarin91 lucarin91 requested a review from dido18 October 17, 2025 15:56
@lucarin91 lucarin91 merged commit 4ddfbf0 into main Oct 17, 2025
6 checks passed
@lucarin91 lucarin91 deleted the add-ci-and-scripts branch October 17, 2025 16:08
lucarin91 added a commit that referenced this pull request Oct 28, 2025
* chore: add ci checks and scripts

* add artifact download in tests

* add missing license

* add gh token

* Exclude go:embed from staticcheck

* Download resources before running the linter

* Never forget the github token

* Update .licensed.yml to comment out reviewed dependencies

* Update license information for multiple dependencies to reflect accurate licensing

* Remove commented-out reviewed dependencies from .licensed.yml

* Refactor Go test workflow to separate orchestrator and package tests

* fix workflows

* fixup! fix workflows

---------

Co-authored-by: MatteoPologruto <m.pologruto@arduino.cc>
Co-authored-by: dido <davideneri18@gmail.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.

3 participants