This repository was archived by the owner on Oct 1, 2024. It is now read-only.
Autogenerate c_cpp_properties.json #1183
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* More documentation on this project * Tried to identify most of the tasks * Added beer money support option
* More investigations and documentation, especially on intrinsic/built-in compiler header include paths
* Added IntelliSense compiler parsing engine code * First injection of the compiler command parser and IntelliSense auto-configuration. Currently injected into "verify" only. * Updated branch documentation to reflect the current state of this project
…n a pull request :)
* Moved pre-build command into separate member function to reduce code replication, better maintainablility and readability * Added pre-build command to "upload using programmer" since it was (probably unintentional) missing there
* Notes regarding settings * Notes where to run the auto-generation
…ration * Added a global configuration switch which allows the IntelliSense auto-configuration to be turned off * Prepared the compiler parser code to be injected into "upload" and "upload using programmer" without overhead * Updated branch documentation
…figuration flag which can override the global flag
* Fixes the line splitting regex as outlined in microsoft#771 * Removed a redundand condition
* Fixed regression introduced with adaptions to latest version of cocopa * Made compile command regex match more stringent
* Added more serial monitor ideas from [John Lonergan](microsoft#463 (comment)) * Added some ideas how to implement a better serial monitor
* More unit testing within cocopa. * Implemented c_cpp_properties merging -> compiler analysis results are merged into existing configuration and will preserve configurations of different name than the vscode-studio default configuration name (currently "Arduino"). This opens up the possibility for users to write their own configurations without having to disable the autogeneration. * Implemented "write on change" - `c_cpp_properties.json` will only be written if a new configuration has been detected.
fearthecowboy
approved these changes
Jan 13, 2021
There was a problem hiding this comment.
I'm feeling pretty good about this.
This PR has done some really good stuff to clean up some quirky looking code. Thanks alot @adiazulay. You've brought you're A game on this one.
|
All the credit goes to @elektronikworkshop @iFreilicht @hlovdal and all the other contributors on the branch for all their hard work! |
This was referenced Jan 20, 2021
|
1st off, this is great! Small recommendation for those who do this but want to do a manual rebuild of c_cpp_properties.json CHANGE: "IntelliSense auto-configuration disabled." TO: "IntelliSense auto-configuration disabled. To manually rebuild your IntelliSense configuration run "Ctrl+Alt+I" |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pr adds autogeneration for c_cpp_properties.json. This will make intelliSense more reliable.
Special thank to @iFreilicht and @hlovdal for rebasing and getting this PR ready to merge.
This PR replaces #1141