hexops/mach

flac: replace decode test with example #958

New issue

Closed

opened 2023-08-09 16:20:41 +00:00 by emidoots · 1 comment

emidoots commented

2023-08-09 16:20:41 +00:00

(Migrated from github.com)

A test which writes out a file to disk, with the only test being the user actually running a separate command afterwards to test it worked, is very bad practice IMO.

A standalone example/ would be better, and then just remove the test.

A test which writes out a file to disk, with the only test being the user actually running a separate command afterwards to test it worked, is very bad practice IMO. A standalone `example/` would be better, and then just remove the test.

emidoots commented

2023-08-12 08:36:22 +00:00

(Migrated from github.com)

Fixed by https://github.com/hexops/mach-flac/pull/12

Fixed by https://github.com/hexops/mach-flac/pull/12

Sign in to join this conversation.

No labels

contributor-friendly

direct3d-headers

driver-os-issue

linux-audio-headers

package-manager

proposal-accepted

vulkan-zig-generated

wayland-headers

xcode-frameworks

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

hexops/mach#958

No description provided.