Feature/viewer 1392 sonar cloud issues on #1400
Conversation
| @@ -0,0 +1,111 @@ | |||
| { | |||
There was a problem hiding this comment.
@nOester was hat es mit diesen Files auf sich?
There was a problem hiding this comment.
@TIL-EBP die kommen aus dem Branch feature/viewer-1344-locale-keys-bereinigen und sind für die Aktualisierung der locales gedacht. Die habe ich wahrscheinlich nicht richtig emerged.
There was a problem hiding this comment.
@nOester Okay vermutlich hast du dann diesen Branch hier auf Basis des feature/viewer-1344-locale-keys-bereinigen Branches erstellt? Gehören die Files hier schon rein? Ansonsten können wir das gerne noch gemeinsam anschauen, wie du die Änderungen wieder rausbekommst
There was a problem hiding this comment.
Hallo @TIL-EBP
Die Files gehören in den feature/viewer-1344-locale-keys-bereinigen. Ja, wäre super, wenn wir das gemeinsam ansehen könnten. Ich schreib dir via Teams.
There was a problem hiding this comment.
@TIL-EBP Könntest Du dir noch mal alle SonarCloud Issues mit dem Tag checkwithdev anshen? Insbesondere, die mit dem Status open
There was a problem hiding this comment.
@nOester ich habe sie mir alle mal angeschaut. Diejenigen in den "swissforage" Files habe ich mal ignoriert, da diese Files ja bald entfernt werden sollen (simmt das noch?). Andere habe ich akzeptiert und bei zweien habe ich Anpassungen am code gemacht und hier gepusht.
There was a problem hiding this comment.
Deine Änderungen schaue ich mir gleich noch an. Dann kannst du von mir aus den PR auch aus dem Draft modus nehmen, wenn du durch bist
SonarCloud: mark as readonly SonarCloud: mark as readonly
nOester
force-pushed
the
feature/viewer-1392-SonarCloud-Issues-ON
branch
from
30842d6 to
6670268
Compare
TIL-EBP
force-pushed
the
feature/viewer-1392-SonarCloud-Issues-ON
branch
from
d6446a8 to
618d58a
Compare
|
Hallo @TIL-EBP
Ich habe mal die "readonly" SonarCloud-Issues bearbeitet.
Könntest Du diese mal ansehen?
Ausserdem habe ich in SonarCloud einige Issues mit dem Tag "checkwithdev" gekennzeichnet. Ich bin ich nicht ganz sicher, wie ich den Code anpassen soll.
Danke und Gruss
Nils