-
-
Notifications
You must be signed in to change notification settings - Fork 476
Remove rogue curly brace in Nano 33 BLE Sense Rev2 IMU accelerator tutorial #2591
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
Remove rogue curly brace in Nano 33 BLE Sense Rev2 IMU accelerator tutorial #2591
Conversation
CLA assistant check
All committers have signed the CLA.
@mzen228 thank you for your contribution! Please sign the CLA so that we can properly review your changes 🙏🏼
@mzen228 thank you for your contribution! Please sign the CLA so that we can properly review your changes 🙏🏼
"Waiting for status to be reported — Contributor License Agreement is not signed yet."
I signed the CLA multiple times but I'm still getting this same message. above what to do now? FYI - my commit username is different from my github account. could this cause the CLA not to go through?
56deca8 to
5eb0b54
Compare
Signed-off-by: mzen228 <mzen228@proton.me>
5eb0b54 to
3636205
Compare
@mzen228 yes, that seems to be the issue - it doesn't seem to understand that you have a github account
I just pushed a rebased history with a username and email consistent with my github account. Now the CLA goes through without a problem!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mzen228 thank you for your keen eyes! Approved and will merge your change now ⭐
Uh oh!
There was an error while loading. Please reload this page.
What This PR Changes
Contribution Guidelines