Skip to content

Add support for "all" host type in package index tool definition #223

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 4, 2021
Merged

Add support for "all" host type in package index tool definition #223

merged 1 commit into from
Aug 4, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 3, 2021

There is a catch-all "all" host type:
https://github.com/arduino/arduino-cli/blob/495a5f1a3d9d2f47b399329d6e6d91562fb683f8/arduino/cores/tools.go#L159

The package index JSON schema was missing support for this host type, resulting in spurious violations of rule IT0016.

@per1234 per1234 added type: bug topic: code Related to content of the project itself labels Aug 3, 2021
@per1234 per1234 requested review from silvanocerza and umbynos August 3, 2021 23:00
There is a catch-all "all" host type:
https://github.com/arduino/arduino-cli/blob/495a5f1a3d9d2f47b399329d6e6d91562fb683f8/arduino/cores/tools.go#L159

The package index JSON schema was missing support for this host type, resulting in spurious violations of rule IT0016.
@per1234 per1234 merged commit 8ec1138 into arduino:main Aug 4, 2021
@per1234 per1234 deleted the add-all-host branch August 4, 2021 08:36
@rsora rsora added the type: imperfection Perceived defect in any part of project label Sep 22, 2021
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants