Fix to never have undefined iteration order and lint against it

[jtran]

Here is the lint: https://rust-lang.github.io/rust-clippy/stable/index.html#/iter_over_hash_type

Now all crates inherit the lints from the workspace.

[qa-wolf]

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
modeling-app	✅ Ready (Inspect)	Visit Preview	Dec 5, 2024 4:17pm

[adamchalmers]

Any reason to prefer IndexMap over BTreeMap?

[jtran]

BTreeMap keeps things in sorted order according to the key type's Ord trait. IndexMap preserves insertion order. Unless we want to sort things, insertion order is a better, more intuitive default, in my opinion. For bindings specifically, it will keep variables in the order that they were defined in the KCL source. That's pretty nice.

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 85.91%. Comparing base (1495cc6) to head (5a4b691).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/wasm-lib/kcl/src/std/sketch.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4665   +/-   ##
=======================================
  Coverage   85.91%   85.91%           
=======================================
  Files          81       81           
  Lines       30117    30117           
=======================================
  Hits        25876    25876           
  Misses       4241     4241           

Flag	Coverage Δ
wasm-lib	85.91% <80.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.