[JENKINS-63580] Update regex for msbuild to recognize linker warnings

[pyatizbyantsevia]

Closes JENKINS-63580

Testing done

Submitter checklist

Edit tasklist title

Beta Give feedback Tasklist Submitter checklist, more options

• Rename
• Copy markdown
• Delete tasklist

Delete tasklist

Delete tasklist block?

Are you sure? All relationships in this tasklist will be removed.

Cancel Delete

1. Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!

   Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
2. Ensure that the pull request title represents the desired changelog entry

   Ensure that the pull request title represents the desired changelog entry
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
3. Please describe what you did

   Please describe what you did
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
4. Link to relevant issues in GitHub or Jira

   Link to relevant issues in GitHub or Jira
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
5. Link to relevant pull requests, esp. upstream and downstream changes

   Link to relevant pull requests, esp. upstream and downstream changes
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
6. Ensure you have provided tests - that demonstrates feature works or fixes the issue

   Ensure you have provided tests - that demonstrates feature works or fixes the issue
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove

Add item to Submitter checklist

[uhafner]

It seems that the regular expression actually is degenerated, too many options have been added year by year. I tried to fix that a little bit, and now your test works :-)

(Maybe it makes sense to start from scratch sometimes... But for now the PR is ready to be integrated, thanks!)

[uhafner]

Can you please remove that file from the change list?

[codecov]

Codecov Report

Merging #952 (df7c0ea) into master (590a415) will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #952      +/-   ##
============================================
+ Coverage     92.94%   93.07%   +0.13%     
- Complexity     2368     2371       +3     
============================================
  Files           347      347              
  Lines          6592     6588       -4     
  Branches        686      686              
============================================
+ Hits           6127     6132       +5     
+ Misses          263      256       -7     
+ Partials        202      200       -2     

Files	Coverage Δ
...a/edu/hm/hafner/analysis/parser/MsBuildParser.java	96.22% <100.00%> (+15.52%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[pyatizbyantsevia]

Thank you very much for making changes to my test, I focused on the parsing problem (on the first assert) and did not bring the test itself to the proper form.
P.s. Before accepting PR, I kindly ask you to put a label (probably HACKTOBERFEST-ACCEPTED) for Hacktoberfest =)

Upd: okay, as I see on the hactoberfest page, this PR has already been accepted and is being processed 😅

[uhafner]

Ok, thanks, released in https://github.com/jenkinsci/analysis-model/releases/tag/v11.9.0