C flavor #3067
C flavor #3067
Conversation
Next commit will build the descriptive items for this flavor.
Adds a new flavor for pure c_cpp developement. This flavor can later add the clang-format and other linters suggested in oxsecurity#763
|
I will still add clang-format. I just wanted to keep my first PR simple as this is my first contribution. |
daltonv
requested review from
nvuillam,
echoix,
bdovaz and
Kurt-von-Laven
as code owners
| @@ -2,6 +2,7 @@ descriptor_id: C | |||
| descriptor_type: language | |||
| descriptor_flavors: | |||
| - cupcake | |||
| - c_cpp | |||
| - dotnet | |||
There was a problem hiding this comment.
I still see a problem here when it comes to clang-format being added. I would naturally put that linter in here and the cpp descriptor file.
However, if I understand it right, that would mean clang-format is also added to the cupcake, dotnet, and dotnetweb linters. I think we only want it in this new c_cpp linter. Right?
There was a problem hiding this comment.
@daltonv you can override descriptor_flavors at linter level, so you could add the following property on clang-format linter definition :)
descriptor_flavors:
- c_cpp
|
/build
|
Proposed Changes
Adds a new c_cpp flavor that can later be used by clang-format (see #763)
Readiness Checklist
Author/Contributor
Reviewing Maintainer
breakingif this is a large fundamental changeautomation,bug,documentation,enhancement,infrastructure, orperformance