Skip to content

Small Improvements - #2

Open
yeganemehr wants to merge 4 commits into
geekbrother:masterfrom
yeganemehr:master
Open

Small Improvements#2
yeganemehr wants to merge 4 commits into
geekbrother:masterfrom
yeganemehr:master

Conversation

@yeganemehr

Copy link
Copy Markdown

Hi Max!
Thank you for your library!
I used it in my project with some improvement & now I'm want to share those with you & others to return the favor.

  • Added user-defined blink speed. (now your library can wink at any speed)
  • Added support for low-active LEDs (like ESP8266 built-in LED)
  • GetState() method modified to return actual state of led (either it's on (true) or off (false))
  • getWinkSpeed() added to get current plan of winking!
  • on() & off() methods added for more convenient.
  • README.md updated according to new modifications.

@geekbrother

Copy link
Copy Markdown
Owner

Hi Amir!
Thank you for your PR and contribution to the open-source community!
I'll add a GitHub CI workflow to check builds automatically and then can merge it.

@geekbrother
geekbrother self-requested a review May 19, 2023 09:23
@geekbrother geekbrother added the enhancement New feature or request label May 19, 2023
@geekbrother geekbrother added this to the 1.0.4 milestone May 19, 2023
@yeganemehr

Copy link
Copy Markdown
Author

Max if you don't have enough time to add CI, I'll be happy to help!
I did recently for one of my libraries:
https://github.com/yeganemehr/cpp-promise/blob/2ccfe0c9d1dfd63fa7c0e09db24f7ca15583af1e/.github/workflows/build-examples.yml
I could also add a CD workflow to upload new versions (releases) to PlatformIO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants