Skip to content

Add Ticker library #1057

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 4, 2018
Merged

Add Ticker library #1057

merged 2 commits into from
Mar 4, 2018

Conversation

bertmelis
Copy link
Contributor

@bertmelis bertmelis commented Jan 31, 2018 •
edited
Loading

see #30

@bertmelis
Copy link
Contributor Author

Hmm, Travis fail because of LED_BUILTIN isn't defined.

@bertmelis
Fix CI (#1)
d2d24f9 
remove LED_BUILTIN in examples
@bertmelis
Copy link
Contributor Author

Travis Fixed 😃

@me-no-dev me-no-dev merged commit 8e29347 into espressif:master Mar 4, 2018
@me-no-dev
Copy link
Member

cool :) Thanks for the contribution! Ticker has been requested for a while

@gjt211
Copy link

gjt211 commented Mar 22, 2018

@bertmelis, @me-no-dev

This is not working for me.
Just updated the core files, the Ticker library folder and examples are there but they won't compile.
Always get fatal error: Ticker.h: No such file or directory
I have restarted the Arduino IDE a few times as well.
Am I doing something wrong?

@gjt211
Copy link

gjt211 commented Mar 22, 2018

@bertmelis, @me-no-dev

Sorry, I renamed the old code library folder but left it in the same place.

/hardware/espressif/esp32
/hardware/espressif/esp32_old

I moved the esp32_old folder outside of the Arduino/hardware folder, restarted the IDE and now it compiles.

@copercini copercini mentioned this pull request Jun 2, 2018
Closed
Curclamas pushed a commit to Curclamas/arduino-esp32 that referenced this pull request Aug 21, 2018
@bertmelis @Curclamas
Add Ticker library (espressif#1057)
58f8c6c 
* Add Ticker library

* Fix CI (#1)

remove LED_BUILTIN in examples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bertmelis @me-no-dev @gjt211