Skip to content

Fix pump noteskins

Fix pump noteskins #73

Triggered via push July 31, 2024 04:55
Status Failure
Total duration 27s
Artifacts

devbuild.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
build: app/src/chart/ChartManager.ts#L1246
Type '{ beat: number; col: number; type: NoteType; }' is not assignable to type 'PartialNotedataEntry | undefined'.
build: app/src/chart/ChartManager.ts#L1306
Type '"Tap"' is not assignable to type '"Hold" | "Roll"'.
build: app/src/chart/ChartManager.ts#L1318
Type '"Tap"' is not assignable to type '"Hold" | "Roll" | undefined'.
build: app/src/chart/ChartManager.ts#L1328
Type '{ beat: number; col: number; type: "Hold" | "Roll" | "Tap"; hold: number | undefined; }' is not assignable to type 'PartialNotedataEntry | undefined'.
build: app/src/chart/ChartRenderer.ts#L350
Argument of type '{ beat: number; col: number; type: NoteType; }' is not assignable to parameter of type 'PartialNotedataEntry'.
build: app/src/chart/gameTypes/common/BasicNotedataParser.ts#L130
Type '{ beat: number; col: number; type: NoteType; }' does not satisfy the expected type 'PartialNotedataEntry'.
build: app/src/chart/gameTypes/common/BasicNotedataParser.ts#L144
Type '{ beat: number; col: number; type: NoteType; }' is not assignable to type 'PartialNotedataEntry | undefined'.
build: app/src/chart/gameTypes/common/BasicNotedataParser.ts#L146
Argument of type '{ beat: number; col: number; type: NoteType; }' is not assignable to parameter of type 'PartialNotedataEntry'.
build: app/src/chart/gameTypes/noteskin/pump/prime/NoteFlash.ts#L16
Cannot find module './Noteskin' or its corresponding type declarations.
build: app/src/data/KeybindData.ts#L12
Cannot find module '../gui/window/NoteskinWindow' or its corresponding type declarations.
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/