Finder tuned: prefiltering for performance#4452
Finder tuned: prefiltering for performance#4452jgrosseo merged 2 commits intoAliceO2Group:devfrom ddobrigk:dev
Conversation
|
@davidrohr @sawenzel If we want analyzers to consider the output of the checkcode test without our help, we have to improve its output. I don't even find the problem in the checkcode output in this PR. Could you let me know? |
|
ps - In case it is a false positive, please merge. |
|
There is already a summary for the checkcode errors. You can find it if you search the log for "list off". But I agree, it would make sense to move this to the beginning of the log.
|
|
So, if I understand correctly this PR failed the o2checkcode because of me not adding underscores before capital letters in the middle of the names of table columns - is that it? ... |
|
@jgrosseo : Like for other builds the output shown is the complete log of the build process (I usually grep for "error"). In order to make this appear at the top or having it stand out, one would need to do adjustments in the CI logic I believe. |
|
@ddobrigk : Yes, the rule for namespaces (as described here https://github.com/AliceO2Group/CodingGuidelines/blob/master/naming_formatting.html) is |
No description provided.