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 CMake modules to generate VSCode Launch and Settings files #11

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

stephendpmurphy
Copy link
Contributor

Pull Request Pre-Flight Checklist

  • Did you add new unit tests for all applicable source code?
  • Did you run through a basic set of functional tests to prove out these changes?
  • If this is a "release" (aka a PR into the main branch), did you increment/set the version in version.h accordingly?

What issues does this close? (The keyword closes followed by a #ISSUE_NUMBER will automatically close the issue when this PR is completed. i.e closes #10)

cmake/vscode_intellisense.cmake Show resolved Hide resolved
cmake/vscode_intellisense.cmake Outdated Show resolved Hide resolved
Copy link

@devinjaenicke devinjaenicke left a comment

Choose a reason for hiding this comment

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

Can you change the target branch to develop?

@@ -74,7 +77,7 @@ project(${PROJECT_NAME} VERSION 1.0 LANGUAGES C)

# Set the C standard and executable suffix
set(CMAKE_EXECUTABLE_SUFFIX ".elf")
set(CMAKE_C_STANDARD 11)
set(CMAKE_C_STANDARD 99)

Choose a reason for hiding this comment

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

Why C 99 instead of C 2011 or C 2018?

Copy link
Contributor Author

@stephendpmurphy stephendpmurphy Nov 19, 2024

Choose a reason for hiding this comment

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

@lukewelchGB Current flags (flags.cmake) are using -std=gnu99 so setting CMake to be in parity.

@stephendpmurphy stephendpmurphy changed the base branch from main to develop November 19, 2024 19:28
@stephendpmurphy
Copy link
Contributor Author

stephendpmurphy commented Nov 19, 2024

@devinjaenicke @lukewelchGB please note I snuck another commit in on you (fb08abd) that reworks the intellisense module a bit and makes it more "auto-magic"

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.

3 participants