Skip to content

Fix CI build #33

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
May 30, 2019
Merged

Fix CI build #33

merged 1 commit into from
May 30, 2019

Conversation

sveljko
Copy link
Contributor

@sveljko sveljko commented May 25, 2019

Don't build M4 board at all - it doesn't have native networking,
test is not useful, yet it fails on ArduinoCI for some unknown
reason.

Don't build ESP32 w/WiFi101.h examples, they are not designed
to work. It was strange they actually worked before.

Don't build M4 board at all - it doesn't have native networking,
test is not useful, yet it fails on ArduinoCI for some unknown
reason.

Don't build ESP32 w/WiFi101.h examples, they are not designed
to work. It was strange they actually worked before.
@vveljko vveljko requested a review from davidnub May 25, 2019 20:22
Copy link

@davidnub davidnub left a comment

Choose a reason for hiding this comment

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

Do we need to keep the m4 and esp32 builds? Just wondering if we should remove the lines vs commenting them out. Other than that, I'm fine with these changes.

@vveljko
Copy link
Contributor

vveljko commented May 30, 2019

Do we need to keep the m4 and esp32 builds?

Arduino is a "moving target". Itself changes, board support changes, libraries change... That's why we keep things in comments, as we might need/want to "enable" them in the future, and it's easier to have them in the comments rather than figuring out what can (or cannot) be put back.

@vveljko vveljko merged commit e11690b into master May 30, 2019
@vveljko vveljko deleted the fix_build branch May 30, 2019 12:23
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.

3 participants