dx-scanner
dx-scanner copied to clipboard
DXHeroes
New Practice: Used Linter for C++
New Practice Proposal
Create a new practice that checks if is used linter for C++.
Sure, no worries! Hope you were doing great in the quizzes ❤️ Let me know if you'll need any help.
@adelkahomolova Ma'am, I resumed my work yesterday, sorry for the delay but semester exams were very brain draining. 😅
I am setting up my environment but there is some issue regarding yarn. How to resolve it?
Also under practices folder what folder name should I put for C++ like CPlusPlus or CPP ?
Hi, @shobhi1310 , I don't know your operating system but the command with npm won't work. You have to follow the installation guide here - https://classic.yarnpkg.com/en/docs/install
Use the CPP as a name for a folder.
@shobhi1310 So choose in the link I sent you and follow the guide :)
Hi, @shobhi1310 , this is not an issue but expected behavior. There are commands you have to use as the help tells you. The command you have to run is yarn start run to run the dx-scanner.
Thank you, ma'am, it runs perfectly now!
Which linter is to be used for C++?
I'm not a C++ developer and choosing the right linter is the part of the task. So you'll have to find it out on your own. Sorry. @shobhi1310 I'm sure you can do it :)
https://github.com/caramelomartins/awesome-linters
I got reference of some widely used C++ linters out of which as per the organization's flavour cppcheck serves the purpose well enough.
There are not only linters, but also some tools for making the code prettier. As I google it, I think you should check these three linters - cppcheck, oclint and clang-tidy. You can get the inspiration from GoLinterUsedPractice to how to check if there is at least one of them. @shobhi1310
In the GoLinterUsedPractice spec file, why has typescript been put for 'If it is not Go' and what should I put for 'If it is not C++' ?
Also, If I need to test it, should I run yarn start run or build using yarn build and then test using yarn test?
It's because this is a test for the case if it's not Go language. So the different language is pasted to the test, so it's testing the method isApplicable() returns false in that case. You can use any of those languages that are not Go. E.g. Typescript.
To run tests, run the command yarn test. You can also specify the path to the test so only tests that you added are running. E.g. yarn test GoLinterUsedPractice.spec.ts. Is it clearer now? @shobhi1310
Ma'am, I got this after running a test on CPPLinterUsedPractise.spec.ts. It has passed the verification tests, but why is the 19.08 s showing red colour?
I will also make a PR today , do once review my code. And, should I also update the readme as well in the Practices section?
It's just the information about the time the tests are running. No worries :) Since there is no Component Detector for C++ yet, you don't have to add the practice to Readme. @shobhi1310
It's just exceeded the Github Rate limit. Nothing to worry about. @shobhi1310
@adelkahomolova Ma'am the purpose of checking practices in C/C++ would not be the same as that of in Go, Typescript, etc. mainly because of package managers not available in C/C++. 🤔
In that case would my PR not be merged? 🙄