Formatting CI + csharpier by JohannHoepfner · Pull Request #192 · Altafraner/afra-app

JohannHoepfner · 2026-02-03T00:37:41Z

TODO

Editorconfig nach unseren Vorstellungen
Evtl ci format workflows jeweils mit build mergen.

rschlossarek · 2026-02-03T08:20:33Z

+            blocksInFutureWithoutEnrollments,
+            wochentyp
+        );
+        foreach (var (_, angebote) in angeboteByBlocks)


Was will der hier? angebote wird genutzt und _ ist ein Discard

+        foreach (var angebot in instanzen)
+        {
+            var belegWunsch = new ProfundumBelegWunsch
            {
-                var belegWunsch = new ProfundumBelegWunsch
-                {
-                    BetroffenePerson = student,
-                    ProfundumInstanz = angebot,
-                    Stufe = stufe,
-                    EinwahlZeitraum = einschreibeZeitraum,
-                };
-                belegWuensche.Add(belegWunsch);
-            }
+                BetroffenePerson = student,
+                ProfundumInstanz = angebot,
+                Stufe = stufe,
+                EinwahlZeitraum = einschreibeZeitraum,
+            };
+            belegWuensche.Add(belegWunsch);
+        }


rschlossarek

Ich hab mal beispielhaft ein paar Dinge markiert, die mich stören. Allgemeines kommt später

rschlossarek · 2026-02-03T08:09:58Z

Können wir automatisch generierte Dateien, wie Migrations, bitte von Formatierungen ausschließen?

Ich sehe den Grund nicht unbedingt, aber ja.

Man könnte die auch beim Generieren direkt formattieren lassen.

rschlossarek · 2026-02-03T08:10:45Z

        UserService userService,
        UserAuthorizationHelper authHelper,
        Guid studentId,
-        bool all = false)


Diese Art Änderung ist so unnötig..

rschlossarek · 2026-02-03T08:11:26Z

+            await authHelper.CurrentUserHasGlobalPermission(GlobalPermission.Otiumsverantwortlich)
+            || await authHelper.CurrentUserHasGlobalPermission(GlobalPermission.Admin);

-        if (!isMentor && !hasBypass) return Results.Unauthorized();


Und die Art Änderung sehe ich auch irgendwie nicht ganz ein, das liegt aber im Diskussionsspielraum

rschlossarek · 2026-02-03T08:13:01Z

        var today = DateOnly.FromDateTime(now);
        var time = TimeOnly.FromDateTime(now);

-        if (!metadata.Verpflichtend || block.SchultagKey != today || time >= metadata.Interval.Start.AddMinutes(30))


können wir hier die erste Zeile direkt hinter das if schreiben und die letzte Klammer mit zur letzten Zeile?

rschlossarek · 2026-02-03T08:13:14Z

        if (await scheduler.CheckExists(jobKey))
            return;

-        var job = JobBuilder.Create<MissingStudentNotificationJob>()


Sehr unnötig..

rschlossarek · 2026-02-03T08:16:48Z

            .Include(t => t.Block)
            .Include(t => t.Otium)
            .Where(t => t.Block.Id == termin.Block.Id && t.Id != terminId)
-            .OrderBy(t => t.OverrideBezeichnung != null ? t.OverrideBezeichnung : t.Otium.Bezeichnung)


Also irgendwie gefällt mir das auch nicht, aber kann verstehen, dass er die Zeile kürzer haben will

rschlossarek · 2026-02-03T08:17:41Z

            .FirstOrDefaultAsync();

        var blockId = fromBlock?.Id ?? toData?.Block.Id;
-        if ((fromBlock is null && fromTerminId != Guid.Empty) || (toData is null && toTerminId != Guid.Empty))


Also zwei Zeilen okay, aber vier??

rschlossarek · 2026-02-03T08:20:33Z

+            blocksInFutureWithoutEnrollments,
+            wochentyp
+        );
+        foreach (var (_, angebote) in angeboteByBlocks)


Was will der hier? angebote wird genutzt und _ ist ein Discard

rschlossarek · 2026-02-03T08:22:34Z

-        : base(message)
-    {
-    }
+        : base(message) { }


Das mag meine IDE gar nicht, kann ich der aber evtl. austreiben

JohannHoepfner added 2 commits February 3, 2026 01:31

Csharpier

457d0a4

Format workflows

f18318b

JohannHoepfner requested a review from rschlossarek February 3, 2026 00:37

github-code-quality bot found potential problems Feb 3, 2026

View reviewed changes

JohannHoepfner changed the title ~~Formatting CI + charpier~~ Formatting CI + csharpier Feb 3, 2026

rschlossarek requested changes Feb 3, 2026

View reviewed changes

JohannHoepfner added 2 commits February 3, 2026 13:46

Webclient formatting ci

3960c81

Backend format using csharpier

c2acaac

JohannHoepfner force-pushed the formatting branch from dad74ec to c2acaac Compare February 3, 2026 12:48

JohannHoepfner force-pushed the main branch 2 times, most recently from 90a9b18 to ecd400c Compare March 10, 2026 07:41

Conversation

JohannHoepfner commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

TODO

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

rschlossarek left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

JohannHoepfner commented Feb 3, 2026 •

edited

Loading