Skip to content

Conversation

@arifbalik
Copy link
Member

@arifbalik arifbalik commented Sep 27, 2024

We wanted to redirect users who wants a debounced driver, as discussed here

Fixes #78813

@zephyrbot zephyrbot added the area: Samples Samples label Sep 27, 2024
@zephyrbot zephyrbot requested review from kartben and nashif September 27, 2024 03:47
@arifbalik arifbalik changed the title samples: button: add comment for input subsys samples: button: add comment for input subsys Sep 27, 2024
@kartben
Copy link
Contributor

kartben commented Sep 27, 2024

Why is the note in the README not enough?

@arifbalik
Copy link
Member Author

arifbalik commented Sep 27, 2024

Apparently people still miss the README file and think there is no debounced driver.

edit: My pr commit body was too long, so i pushed with a smaller one

@arifbalik arifbalik force-pushed the basic_button_example_doc_update branch from c24ae0c to d779c80 Compare September 27, 2024 11:32
@kartben kartben requested a review from fabiobaltieri October 25, 2024 10:10
Copy link
Member

@fabiobaltieri fabiobaltieri left a comment

Choose a reason for hiding this comment

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

Thanks!

@arifbalik
Copy link
Member Author

@fabiobaltieri the file has the copyright line but the CI somehow fails with this output

* scan/tests/subsys/jwt/src/tls_config/user-tls-conf.h missing copyright.

We wanted to redirect users who wants a debounced driver zephyrproject-rtos#78813

Signed-off-by: Arif Balik <arif.balik@enda.com>
@fabiobaltieri fabiobaltieri force-pushed the basic_button_example_doc_update branch from d779c80 to 517f09a Compare October 25, 2024 13:20
@fabiobaltieri
Copy link
Member

@arifbalik yeah kicked a retry but it failed still, the file is gone from main but I think that that specific check does not rebase, I rebased and pushed it for you, should work now. No idea how it got there in the first place and why it's failing here though but let's pretend it was transient.

@fabiobaltieri fabiobaltieri added this to the v4.0.0 milestone Oct 25, 2024
@arifbalik
Copy link
Member Author

thanks, this will be a fun pr to merge on my bday today 🎉 😄

@dkalowsk
Copy link
Contributor

happy birthday...

@dkalowsk dkalowsk merged commit 25272ae into zephyrproject-rtos:main Oct 26, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Button example is not using gpio-keys driver

5 participants