Add Portenta X8 to compilation checks#76

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

aliphys wants to merge1 commit intoarduino:main

base:main

Choose a base branch

fromaliphys:portentaX8CICoverage

Open

Add Portenta X8 to compilation checks#76

aliphys wants to merge1 commit intoarduino:mainfromaliphys:portentaX8CICoverage

Conversation

Copy link

aliphys commentedMay 22, 2024

Partlycloses#75

ci: add Portenta X8 to compilation checks

20cd94d

per1234 requested changes

May 22, 2024

View reviewed changes

Copy link

Contributor

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 usesSerial won't compile for the Portenta X8. So the workflow must be configured to not compile those sketches for Portenta X8.

Please create aserial board attribute and move the sketches that useSerial fromjobs.build.env.UNIVERSAL_SKETCH_PATHS to the newly createdserial element injobs.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)