Sonar: %n should be used in place of \n to produce the platform-specific line separator.

[qmonmert]

Fix https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&id=jhipster_jhipster-lite&open=AY32YPdwSjlKq-G0oiu5&tab=code

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (7513cfa) to head (4b5a0f0).
Report is 10 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##                main     #9090   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
+ Complexity      2920      2918    -2     
===========================================
  Files            733       733           
  Lines          12616     12611    -5     
  Branches         254       254           
===========================================
- Hits           12616     12611    -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[qmonmert]

@murdos \\n are really necessary in the regex because tests are ✅ without?

[murdos]

@murdos \\n are really necessary in the regex because tests are ✅ without?

@qmonmert : Good point. I was not aware until now, but \s match return line \n, so line separator is indeed not necessary.
I've pushed to your branch a modification of an existing test using a dependency with multiple exclusions, to be sure that this case is covered.

So +1 for removing the line separator :-)

[qmonmert]

@murdos seems good

[murdos]

Thanks @qmonmert ! 🙂

[qmonmert]

@murdos @pascalgrimaud ✅
https://sonarcloud.io/summary/overall?id=jhipster_jhipster-lite