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

The nrf52x submodule is severely outdated #8

Open
lhstrh opened this issue Aug 15, 2022 · 2 comments
Open

The nrf52x submodule is severely outdated #8

lhstrh opened this issue Aug 15, 2022 · 2 comments

Comments

@lhstrh
Copy link

lhstrh commented Aug 15, 2022

Why?

@brghena
Copy link
Member

brghena commented Aug 16, 2022

I suspect it never had major improvements that were necessary to pull into the labs. Tracking the head of nrf52x-base always has the chance of breaking things, and when the labs are already working it's not really necessary. Maybe others who taught EE149 more recently have additional information though.

I think now might be a good time, if you're otherwise making changes and testing things anyways. It might be worth considering an SDK update too. I've got a similar software setup (different hardware) for my class at Northwestern which is on SDK 16 and a March 2021 version of nrf52x-base https://github.com/nu-ce346/nu-microbit-base (Note that there's an SDK 17 version as well, which I don't think anyone with nrf52x-base is using yet.)

@lhstrh
Copy link
Author

lhstrh commented Aug 19, 2022

Thanks for the explanation, @brghena! I guess the trouble is that there are no regression tests, so the only way to find out whether something is broken is to do all the exercises...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants