[PWGLF] logic flow uniformed to nucleiSpectra for cross check #15623
+81
−67
Annotations
6 errors, 1 warning, and 2 notices
|
Run tests
Process completed with exit code 1.
|
|
Run tests
Issues have been found.
|
|
Run tests:
PWGLF/TableProducer/QC/nucleiQC.cxx#L132
Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
|
|
Run tests:
PWGLF/TableProducer/QC/nucleiQC.cxx#L111
Use lowerCamelCase for names of configurables and use the same name for the struct member as for the JSON string. (Declare the type and names on the same line.)
|
|
Run tests:
PWGLF/TableProducer/QC/nucleiQC.cxx#L1
Name of a workflow file must match the name of the main struct in it (without the PWG prefix). (Class implementation files should be in "Core" directories.)
|
|
Run tests:
PWGLF/TableProducer/QC/nucleiQC.cxx#L193
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
Complete job
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: thollander/actions-comment-pull-request@v3. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Run tests
To tolerate certain issues in a directory, add a line with the test name in "o2linter_config".
|
|
Run tests
Exceptionally, you can disable a test for a line by adding a comment with "o2-linter: disable=" followed by the name of the test and parentheses with a reason for the exception.
|
Loading