Update calculation of workspace files

antiagainst / codeclimate-cppcheck

Code Climate Engine for Cppcheck

MIT License

27 stars 17 forks source link

Closed dblandin closed 6 years ago

dblandin commented 6 years ago

Extract file workspace calculation and use os.walk.

dblandin commented 6 years ago

I believe this addresses https://github.com/antiagainst/codeclimate-cppcheck/issues/2 but it looks like cppcheck will emit issues for header files, even if the header paths are not included in the provided file list.

Trying to figure out a way around this...

antiagainst commented 6 years ago

The code LGTM now, but I haven't checked it on CodeClimate CLI. Do you want to solve the problem in another PR? I can merge it right now.

dblandin commented 6 years ago

@antiagainst Yep, feel free to merge. I'll open up another PR with some updates.