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

📝 Black Pill / I3DBEEZ9 Boards #502

Closed

Conversation

thisiskeithb
Copy link
Member

Add latest boards.

@thisiskeithb
Copy link
Member Author

I like that we have access to update the boards list now, but I'm still thinking this could be automated. Perhaps pulling a copy of upstream boards.h and parsing it on deployment?

I'll have to get this building locally and see what can be scripted through GitHub Actions.

@thinkyhead thinkyhead force-pushed the master branch 2 times, most recently from beb10cb to 52b43fd Compare June 8, 2023 07:14
@thisiskeithb
Copy link
Member Author

Committed directly: 52b43fd

@thisiskeithb thisiskeithb deleted the pr/add_latest_boards branch June 8, 2023 22:27
@thinkyhead
Copy link
Member

I like that we have access to update the boards list now, but I'm still thinking this could be automated.

Automation is certainly possible, but I haven't gotten around to it with all the other junk I need to attend to. My strategy meanwhile is to occasionally look at the commit history of boards.h in reverse chronological order and add boards until I reach the last one that was already added.

@thisiskeithb
Copy link
Member Author

thisiskeithb commented Aug 7, 2023

My strategy meanwhile is to occasionally look at the commit history of boards.h in reverse chronological order and add boards until I reach the last one that was already added.

As someone who frequently adds boards & deals with pins files a lot, I don’t have a problem with submitting PRs like I’ve done here & in #514.

@thinkyhead
Copy link
Member

You don't need to go to so much trouble. This is something I will do in the general course and certainly at the time of any tagged release.

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

Successfully merging this pull request may close these issues.

2 participants