search

22d3e06 / instalist-server

Server side for synchronizing data from InstaList.
Apache License 2.0
0 stars 2 forks source link

No SonarQube usage #2

Closed hdorottya closed 7 years ago

hdorottya commented 7 years ago

A build.gradle manipulációjával kijátszottam a pom.xml generálását a SonarQube működéséhez, melynek hiányára az eszköz használat közben felhívta a figyelmet. Az ellenőrzés eredménye nem mutatott bug-ot, illetve sérülékenységet, viszont számos code smell-re és duplikációra hívta fel a figyelmet a projektben. Ezután ezek egy részét javítottam ki!

hdorottya commented 7 years ago

unused import removed

hdorottya commented 7 years ago

block of commented-out lines of code removed

hdorottya commented 7 years ago

useless parenthesis removed

hdorottya commented 7 years ago

unused imports removed

hdorottya commented 7 years ago

block of commented-out lines of code removed

hdorottya commented 7 years ago

block of commented-out lines of code removed

hdorottya commented 7 years ago

modify build.gradle to avoid pom.xml

hdorottya commented 7 years ago

block of commented-out lines of code removed

hdorottya commented 7 years ago

block of commented-out lines of code removed

hdorottya commented 7 years ago

useless parenthesis removed

hdorottya commented 7 years ago

useless parenthesis removed

hdorottya commented 7 years ago

useless parenthesis removed

hdorottya commented 7 years ago

unused imports removed

hdorottya commented 7 years ago

unused imports removed

hdorottya commented 7 years ago

unused imports removed

hdorottya commented 7 years ago

Wiki oldal: https://github.com/22d3e06/instalist-server/wiki/SonarQube

ofalvai commented 7 years ago

Én szívesen leellenőrzöm. A beüzemelésnél volt valami extra lépés, amire figyeljek? Ha igen, beleírjuk a wikibe?

hdorottya commented 7 years ago

Arra figyelj, hogy a build.gradle-be bekeruljon az elejere a plugin, amit commitoltam. Ezzel kikeruljuk a pom.xml hianyat. En linuxon csinaltam, ott ennyi volt es magat az analitikat a 'gradle sonarqube' paranccsal tudtam ott inditani, mert ugye a gradle valtoztattam. Ha windowson csinalod akkor a conf mappaban a konfiguracios fajlban a path-nal meg kell adnod a jvm elerest kulonben nem fog felallni a sonarqube

hdorottya commented 7 years ago

Beleirhatom a wikis oldalaba ha gondoljatok ezt

ofalvai commented 7 years ago

Javítottam code smell-eket:

És frissítettem a wiki oldalát.

micskeiz commented 7 years ago

OK.

A javításoknál arra figyeljetek azért, hogy ha a projekt használ konzisztensen egy adott elnevezési konvenciót (pl. lokális változónál _ prefix), akkor hiába javasol a SonarQube mást, nem biztos, hogy érdemes átírni a teljes projektet. Ez az eredeti fejlesztő döntése, hogy milyen elnevezési konvenciót szeretne használni.