adafruit / circuitpython

CircuitPython - a Python implementation for teaching coding with microcontrollers
https://circuitpython.org
Other
4.07k stars 1.2k forks source link

robohatmm1 in 4.1.x branch needs to be renamed to robohatmm1_m4 to match master #2412

Closed makermelissa closed 4 years ago

makermelissa commented 4 years ago

This is causing an issue on the build names and how the boards show up on circuitpython.org

makermelissa commented 4 years ago

Unfortunately, I'm not the greatest at dealing with multiple branches on such a complex repo, so I'm not sure the best way to go about it.

dhalbert commented 4 years ago

So after it's fixed manually by https://github.com/adafruit/circuitpython-org/pull/344, we need to rename the board before doing any more 4.x releases (per discussion with @makermelissa in discord).

makermelissa commented 4 years ago

It might need to be fixed before the next 5.x release if the data files are completely regenerated.

dhalbert commented 4 years ago

I can add renamed assets in the 4.1.2 release to match the new board name, so that the links will continue to work. Where are the board names extracted from: the last release to generate the circuitpyhon-org PR, or somewhere else?

makermelissa commented 4 years ago

You don't need to rename the assets. I'm pointing to the current ones.

makermelissa commented 4 years ago

I think you just need to change the folder name under ports/atmel-samd/boards

dhalbert commented 4 years ago

@wallarug See above. The robohatmm1 in the 4.x tree is a SAMD21 board. Is that discontinued? Are you expecting everyone to use CircuitPython 5.x now? If that's true, we could just remove the board from 4.x. Thanks for looking.

wallarug commented 4 years ago

Hey @makermelissa and @dhalbert

Please remove robothatmm1 from the 4.x tree. We are only making the SAMD51.

If you feel so inclined, the robohatmm1_m4 from the master branch can be put onto 4.x as well.

AFAIK from when I last touched this there was no build issues with robohatmm1_m4 on any of the Circuit Python repositories.

wallarug commented 4 years ago

Can this issue be closed now? @dhalbert @makermelissa

makermelissa commented 4 years ago

Yeah, I'm good with closing it.

dhalbert commented 4 years ago

Fixed by #2415.