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

Add Portenta X8 to compilation checks #76

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aliphys
Copy link

@aliphys aliphys commented May 22, 2024

Partly closes #75

Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

As you reported at #74, any sketch that uses Serial won't compile for the Portenta X8. So the workflow must be configured to not compile those sketches for Portenta X8.

Please create a serial board attribute and move the sketches that use Serial from jobs.build.env.UNIVERSAL_SKETCH_PATHS to the newly created serial element in jobs.build.strategy.matrix.include[]

In case you don't know how this system works, I provided some explanation here:

arduino-libraries/Arduino_UnifiedStorage#4 (comment)

@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ACELL04] Include the Portenta X8 and Nicla Sense ME as part of the CI workflow
2 participants