Skip to content

pre match#22

Merged
yuval20100 merged 19 commits intomasterfrom
scouting/feat/pre-match
Feb 4, 2026
Merged

pre match#22
yuval20100 merged 19 commits intomasterfrom
scouting/feat/pre-match

Conversation

@yuval20100
Copy link
Contributor

No description provided.

@yuval20100 yuval20100 force-pushed the scouting/feat/pre-match branch 3 times, most recently from ae60d2e to 5cfcf20 Compare January 27, 2026 11:09
@yuval20100 yuval20100 force-pushed the scouting/feat/pre-match branch from 8476597 to 0019577 Compare February 1, 2026 09:35
Copy link
Contributor

@YoniKiriaty YoniKiriaty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merge master to you

@YoniKiriaty YoniKiriaty added the feature A new Feature! label Feb 1, 2026
@@ -0,0 +1,93 @@
//בס"ד
import React, { type FC, type Dispatch, type SetStateAction } from "react";

Check warning

Code scanning / ESLint

Disallow unused variables Warning

'React' is defined but never used.
@@ -0,0 +1,93 @@
//בס"ד
import React, { type FC, type Dispatch, type SetStateAction } from "react";

Check warning

Code scanning / ESLint

Disallow unused variables Warning

'Dispatch' is defined but never used.
@@ -0,0 +1,93 @@
//בס"ד
import React, { type FC, type Dispatch, type SetStateAction } from "react";

Check warning

Code scanning / ESLint

Disallow unused variables Warning

'SetStateAction' is defined but never used.
Comment on lines +75 to +78
type: event.target.value as
| "practice"
| "qualification"
| "playoff",

Check warning

Code scanning / ESLint

Disallow type assertions that narrow a type Warning

Unsafe type assertion: type '"practice" | "qualification" | "playoff"' is more narrow than the original type.
@yuval20100 yuval20100 merged commit e95deb2 into master Feb 4, 2026
2 checks passed
@yuval20100 yuval20100 deleted the scouting/feat/pre-match branch February 4, 2026 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new Feature!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants