Skip to content
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

ci: add more targets to .eil.yml #484

Open
1 task done
trombik opened this issue Dec 1, 2022 · 2 comments
Open
1 task done

ci: add more targets to .eil.yml #484

trombik opened this issue Dec 1, 2022 · 2 comments
Assignees
Labels
ci Continuous integration

Comments

@trombik
Copy link
Collaborator

trombik commented Dec 1, 2022

The issue

the CI includes other targets (see 4be76b0). these new targets are not tested by CI because .eil.yml does not include the targets.

The context

the components in the repository are mostly target-independent. there are few differences we depend on, such as macros definitions like SPI name, number of available channels. as such, the components should be marked as "supported" by default.

if a component does not work because of differences in a target (and cannot be fixed by the component), the target will be removed from .eil.yml.

Possible solution

update .eil.yml with the new targets.

Confirmation

  • This report is not a bug report in code, a question, nor a request for drivers.
@trombik trombik self-assigned this Dec 1, 2022
@trombik trombik changed the title add more targets to .eil.yml bugfix: add more targets to .eil.yml Dec 1, 2022
@trombik trombik changed the title bugfix: add more targets to .eil.yml ci: add more targets to .eil.yml Dec 1, 2022
@trombik
Copy link
Collaborator Author

trombik commented Dec 1, 2022

the fix should include:

  • a section that describes this policy in CONTRIBUTING.md
  • updates to README.yml (supported targets)

@trombik
Copy link
Collaborator Author

trombik commented Dec 14, 2022

see also #488. the helper needs modifications, too.

@trombik trombik added the ci Continuous integration label Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
Development

No branches or pull requests

1 participant