search

secularbird / cpplint-extension

vscode cpplint extension
MIT License
22 stars 12 forks source link

Add the 'root' and 'repository' agruments to configuration #7

Closed Bulmanator closed 6 years ago

Bulmanator commented 6 years ago

This extension works great, however I would like to be able to pass the --root argument to define where my header guards are supposed to start (otherwise the linter picks them up as being incorrect). Also the repository argument would be nice to be able to set as "${workspaceFolder}" or something in the case that version control is not being used.

secularbird commented 6 years ago

version 0.0.8 has updated, add these settings

Bulmanator commented 6 years ago

Thank you for adding these settings, however for some reason the root option does not seem to be working for me. It still asks me to append INCLUDE to the front of my header guards. This happens with both the default and cpplint.root=include.

Bulmanator commented 6 years ago

Never mind, you can ignore my last comment and I can now confirm that these options are working correctly. Thanks again for implementing them