feat: add TeamCity output format

[ferhatelmas]

fixes #3597

[ldez]

LGTM

[alexandear]

Did you test these changes on a TeamCity server?

I did. And it doesn't work. My steps:

1. Created a free TeamCity Cloud instance
2. Set a new project with build configuration which:

• build golangci-lint binary in this branch:

cd test
make -C ".." build

• runs golangci-lint inside test/testdata directory with --out-format=team-city:

LINTERS=("govet" "errcheck" "staticcheck" "unused" "gosimple" "ineffassign" "asasalint" "wrapcheck" "nilerr" "nlreturn" "gosec")

cat > severity.yaml << EOF
severity:
  default-severity: info
  rules:
    - linters:
        - govet
      severity: warning
    - linters:
        - unused
      severity: error
    - linters:
        - gosimple
      severity: weak warning
EOF

for i in "${!LINTERS[@]}"
do
   LINTER="${LINTERS[$i]}"
   ./../../golangci-lint run --go=1.17 --config=severity.yaml --allow-parallel-runners --disable-all --max-same-issues=100 --out-format=team-city -Etypecheck -E"$LINTER" "$LINTER.go"
done

Output: "Code Inspection" tab is empty.

You could view this directly on the TeamCity (use "Log in as guest").

[alexandear]

So, I pushed the commit f40b25d with fixes.

Changes:

• print SEVERITY instead of additional attribute;
• populate inspectionType from linter's Name and Desc;
• test escaping characters;
• unexport inspectionType and inspectionInstance;
• extend .golangci.reference.yml severity section with TeamCity;
• minor documentation fixes.

And it now works:

See here (use "Log in as guest").

[ldez]

Did you test these changes on a TeamCity server?
I did. And it doesn't work.

No, and this is why I asked you to test it and why I didn't merge before your review.

So, I pushed the commit with fixes.

I also asked you to not add commits to the PR.

[alexandear]

I also asked you to not add commits to the PR.

Ok, will do that in future PRs. Sorry, I didn't see the comment. Maybe because it was resolved and GitHub doesn't send notifications in such cases.