Skip to content
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

Add new board BastBLE #3662

Merged
merged 9 commits into from
Dec 22, 2020
Merged

Add new board BastBLE #3662

merged 9 commits into from
Dec 22, 2020

Conversation

Soulkrown
Copy link

Hi
I would like to be able to add this new BastBLE card from electronic cats to your repository.
I await your feedback and response.
I thank you in advance.
Best regards

@tannewt tannewt added this to the Long term milestone Nov 9, 2020
@tannewt tannewt self-requested a review November 9, 2020 23:27
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Please follow up when the CI fails. Thanks! I don't see any other potential issues.

.github/workflows/build.yml Outdated Show resolved Hide resolved
@tannewt
Copy link
Member

tannewt commented Nov 10, 2020

I suggest installing pre-commit locally to test more rapidly: https://pre-commit.com/#installation

@sabas1080 sabas1080 requested a review from tannewt November 17, 2020 03:21
@tannewt
Copy link
Member

tannewt commented Nov 17, 2020

The bastble build is broken. Looks like it needs a function name updated: https://github.com/adafruit/circuitpython/pull/3662/checks?check_run_id=1410247014

@tannewt tannewt added the board New board or update to a single board label Dec 1, 2020
@sabas1080
Copy link
Collaborator

@tannewt ready, thanks!

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Looks good! One comment that you can make a follow up PR for if you like.

{ MP_ROM_QSTR(MP_QSTR_AREF), MP_ROM_PTR(&pin_P0_31) },

// voltage sense battery
{ MP_ROM_QSTR(MP_QSTR_VBAT), MP_ROM_PTR(&pin_P0_26) },
Copy link
Member

Choose a reason for hiding this comment

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

Note, most boards name this pin VOLTAGE_MONITOR or BATTERY.

@tannewt tannewt merged commit fb0c74c into adafruit:main Dec 22, 2020
@sabas1080 sabas1080 deleted the bastble branch September 9, 2021 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board New board or update to a single board enhancement nordic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants