Updating Scarb references to latest v2.8.1

[stranger80]

This is a naive attempt to fix #216 - as without these Starklings refuse to compile.

[enitrat]

@stranger80 thank you for updating starklings 🙏 can you update rust in CI to 1.80.0 ?

[stranger80]

Rust updated to 1.80.0 in workflow definition.

[shramee]

Build seems to be failing does it work when building/running locally?

[enitrat]

hm apparently 1.80.0 fials as well
i tried with 1.82.0-nightly and it works

[stranger80]

It works for me with 1.80.1 - shall we use that?

[enitrat]

yes we can

[shramee]

I'll leave this to you @enitrat

[stranger80]

OK, one more step was missing - force update the dependencies (cargo update) - this is required to get to the latest crate versions.

[shramee]

@enitrat This PR seems to have removed the build step in ci...

[enitrat]

Ah oops I didn't see the last commit 🤦

[shramee]

Aah actually, looks like CI is broken...

[shramee]

Addressed in #220

[shramee]

@enitrat We might have to undo this one, because we need Cairo v2.8 compatibility... Even after updating versions on runner crates... I'm getting this error...

So tests don't work. Probably some changes in Serde with the new Scarb version...

[enitrat]

ah ok yes sorry 🙏 did this on mobile, let's revert

[shramee]

No worries, because of the broken CI file, the tests weren't run... So there weren't any errors on the PR.

[enitrat]

We can just add cfg(test) attribute on test modules I think

[shramee]

Didn't quite work...

It does work via scarb in the runner crate though... But not via Starklings.

[shramee]

Fixed serde...
#221 (comment)