Skip to content

Commit 0ef75ea

Browse files
committed
fix: address review feedback
1 parent faa7bc4 commit 0ef75ea

4 files changed

Lines changed: 2 additions & 6 deletions

File tree

.gitignore

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,3 @@ frontend/static/webfonts-preview
133133
frontend/.env.sentry-build-plugin
134134
.claude/worktrees
135135
1024MiB
136-
.scannerwork/
137-
.claude/
138-
sonar-project.properties

backend/src/dal/connections.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@ export async function aggregateWithAcceptedConnections<T>(
331331

332332
function getKey(initiatorUid: string, receiverUid: string): string {
333333
const ids = [initiatorUid, receiverUid];
334-
ids.sort((a, b) => a.localeCompare(b));
334+
ids.sort();
335335
return ids.join("/");
336336
}
337337

frontend/src/ts/modals/user-report.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ export async function show(options: ShowOptions): Promise<void> {
7272
},
7373
});
7474

75-
new CharacterCounter(modal.getModal().qsr(".comment"), 250); // NOSONAR - constructor registers DOM listeners as side effect
75+
new CharacterCounter(modal.getModal().qsr(".comment"), 250);
7676
}
7777

7878
async function hide(): Promise<void> {

frontend/src/ts/utils/misc.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -539,7 +539,6 @@ export function promiseWithResolvers<T = void>(): {
539539

540540
const promiseLike = {
541541
// oxlint-disable-next-line no-thenable promise-function-async require-await
542-
// NOSONAR
543542
async then<TResult1 = T, TResult2 = never>(
544543
onfulfilled?: ((value: T) => TResult1 | PromiseLike<TResult1>) | null,
545544
onrejected?:

0 commit comments

Comments
 (0)