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 building hello world on all platforms as part of sanitycheck #221

Closed
wants to merge 1 commit into from

Conversation

galak
Copy link
Collaborator

@galak galak commented May 17, 2017

Do a simple build test on all platforms to make sure they build when we
change things. This will hopefully catch an errors we run into from
time to time on platforms that don't get typicaly run by sanitycheck.

Signed-off-by: Kumar Gala [email protected]

Do a simple build test on all platforms to make sure they build when we
change things.  This will hopefully catch an errors we run into from
time to time on platforms that don't get typicaly run by sanitycheck.

Signed-off-by: Kumar Gala <[email protected]>
@galak galak requested a review from nashif May 17, 2017 14:18
@nashif
Copy link
Member

nashif commented May 17, 2017

This is already covered in #8 btw, and in a more flexible and configurable way. Right now I just need to go and add latest boards and s/TBD/something useful/

@galak
Copy link
Collaborator Author

galak commented May 17, 2017

Do we want this as a stop gap until we get the YAML stuff merged in? Otherwise I can drop it.

@nashif
Copy link
Member

nashif commented May 17, 2017

Do we want this as a stop gap until we get the YAML stuff merged in? Otherwise I can drop it.

This is overriding the test reports now, so it is a NO GO in the current form. Need to change sanitycheck to create a new xml output and append. Let me spend some time on the YAML stuff today and also address the test report issue before we go with this as a stop gap.

@galak
Copy link
Collaborator Author

galak commented May 17, 2017

Sounds good, wasn't sure about the output and how this impacted things.

@nashif
Copy link
Member

nashif commented Jun 22, 2017

not needed anymore, we do now have 'build_on_all' keyword which causes a test to be built on ALL available boards.. (as part of #8 )

@nashif nashif closed this Jun 22, 2017
@galak galak deleted the ci branch June 22, 2017 20:57
nagineni pushed a commit to nagineni/zephyr that referenced this pull request Nov 20, 2017
…ature

[Demo] Fix the temperature display to use integer
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.

2 participants