Fix for 'cargo test'

[id3v1669]

pr to fix issue #113

[Decodetalkers]

to freeze-feat-andreas branch, that is the developing branch

[Decodetalkers]

can you rebase it? thanks

[id3v1669]

can you rebase it? thanks

I'm working on a new commit as in branch freeze-feat-andreas a lot more errors during cargo test. I'll ping you when done.

[id3v1669]

@Decodetalkers Done. Added also ignore to pixel-art in region.rs as it was also handled as error by cargo test Bumped flake.lock due to error:

error: package clap_derive v4.5.3 cannot be built because it requires rustc 1.74 or newer, while the currently active rustc version is 1.70.0

[Decodetalkers]

@Decodetalkers Done. Added also ignore to pixel-art in region.rs as it was also handled as error by cargo test Bumped flake.lock due to error:

error: package clap_derive v4.5.3 cannot be built because it requires rustc 1.74 or newer, while the currently active rustc version is 1.70.0

maybe you can update nix, to which has higher version of rust

[id3v1669]

maybe you can update nix, to which has higher version of rust

That's why I bumped flake.lock file, so newer version of cargo is used now cargo 1.76.0.

[Shinyzenith]

I think this is a good time to run cargo test during builds even though tests don't exist. This helps us avoid such issues further down the line and we do have a WIP pr for integration testing in the future. How do you feel about this change? @id3v1669

[id3v1669]

I think this is a good time to run cargo test during builds even though tests don't exist. This helps us avoid such issues further down the line and we do have a WIP pr for integration testing in the future. How do you feel about this change? @id3v1669

Sounds great, should I add cargo test into Makefile in this pr or you will do that later in pr for integration testing?

[Shinyzenith]

Feel free to add it to this pr itself. Since it's a fairly minor change.