Skip to content

Very Special Code#120

Merged
mjh9585 merged 9 commits into
mainfrom
feature/mjh9585/vscode-support
Sep 27, 2025
Merged

Very Special Code#120
mjh9585 merged 9 commits into
mainfrom
feature/mjh9585/vscode-support

Conversation

@mjh9585

@mjh9585 mjh9585 commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

Adds default VScode configurations to support easy development withing VScode. This uses the collection of STM32Cube extensions for VScode to provide building and debugging support without the need for OpenOCD. Supports STM32F302, STM32F334, and STM32F446.

  • Includes a set of recommend extensions to install when repository is cloned
  • Adds default launch config
  • Adds set of task for building, cleaning, and running code
  • Adds a set of quick buttons to the status bar for running tasks
  • Adds a set of CMakePresets for all of the different boards

@mjh9585 mjh9585 requested review from a team and aclowmclaughlin August 25, 2025 00:04

@DiegoLHendrix DiegoLHendrix left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good job

@ActuallyTaylor ActuallyTaylor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this is a great idea :) I have some questions about some included file changes, and a few about the config.

Comment thread include/core/manager.hpp
Comment thread samples/timer/main.cpp
Comment thread CMakePresets.json Outdated
Comment thread .vscode/tasks.json Outdated
@mjh9585 mjh9585 requested a review from ActuallyTaylor August 27, 2025 03:39

@aclowmclaughlin aclowmclaughlin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There's some unrelated changes that I'd prefer be in a different pr, but otherwise I mean yeah sure looks good ig.

Comment thread include/core/manager.hpp Outdated
Comment thread samples/timer/main.cpp Outdated

@ActuallyTaylor ActuallyTaylor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good :)

@aclowmclaughlin aclowmclaughlin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fax my guy spit your shit indeed.

@mjh9585 mjh9585 force-pushed the feature/mjh9585/vscode-support branch from 1c40e13 to 2a65f52 Compare September 27, 2025 16:08
@mjh9585 mjh9585 merged commit 3c58c76 into main Sep 27, 2025
1 check passed
@mjh9585 mjh9585 deleted the feature/mjh9585/vscode-support branch September 28, 2025 02:50
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

Successfully merging this pull request may close these issues.

5 participants