Repository navigation
Upgrade eslint - #158
Upgrade eslint#158
Conversation
| const { loseStreak, results, winStreak } = latestPlayerResults || {}; | ||
| const streak = winStreak || loseStreak || undefined; | ||
| const { loseStreak, results, winStreak } = latestPlayerResults ?? {}; | ||
| const streak = winStreak ?? loseStreak ?? undefined; |
There was a problem hiding this comment.
Er dette korrekt..
Hvis winstreak er 0 og loseStreak er 3.. Så bliver din streak nu til 0 right?
There was a problem hiding this comment.
Burde have været en lose streak right?
There was a problem hiding this comment.
Det er ikke korrekt, godt fanget. Jeg har nu omskrevet så latestPlayerResults slet ikke kan være null længere 👌
dc2f7bc to
9ac61ea
Compare
9ac61ea to
d2810a1
Compare
|
Jeg tror ikke det har noget med dine rettelser at gøre mere at seeding seeder med "" som picture. |
oh great, I was sure I had updated this to have a default image from seeding 🤔 anyway I confirmed its the same problem on the main branch, so out of scope for here :D (jeg ser en engelsk youtube video, så derfor skifter jeg lige sprog 😅 ) |
This branch was previously deployed
2 participants
Jeg har længe haft en fejl med at eslint +9 krævede det nye flatpack format, selvom vi brugte 8.x.x. Så nu har jeg opdateret
Der var nogle template specifikke regler som jeg tror engang er blevet kopieret men som hvor de templates aldrig har eksisteret i vores projekt. Så har slettet det.
Vi ignorerede en fejl-type med en kommentar på at den ikke virkede ordentligt. Jeg tror den virker nu. Så vi ignorerer den ikke længere.
Kan du huske jeg har haft et problem hvor min PC formaterede koden lidt anderledes end vores CI tjek? Af en eller anden grund opfører min PC sig anderledes med denne opdatering af ESLint 🤷♀️