Cut release v0.25.4

[Irev-Dev]

What's Changed

• KCL Breaking change: Circle function arguments have changed fromcircle([yourCenterX, yourCenterY], yourRadius, sketch, tag?) to circle({ center: [yourCenterX, yourCenterY], radius: yourRadius }, sketch, tag?) (Circle #3860)
• Circle now has a point-and-click tool while sketching! (Circle #3860)

Full Changelog: v0.25.3...v0.25.4

[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	Sep 23, 2024 1:35pm

[franknoirot]

MacOS smoke test is good except one thing.

I get Permission denied when trying to run the app with the following command: /Applications/Zoo\ Modeling\ App.app ~/Documents/kcl-samples/ball-bearing/ball-bearing.kcl. If I prepend open to that, it runs fine. I upgraded to MacOS Sequoia last night so I'm not sure if that had an impact on how permissions work.

[franknoirot]

Windows smoke test behaved well other than opening multiple files via double-click. I believe this is relatively untested behavior on Windows, so I've made a new issue with #3955, and I don't believe it should block release.

[franknoirot]

Minor issues with MacOS and Windows found, but no showstoppers.

[franknoirot]

When @pierremtb is back from vacation I'll ask him more about how the updater test is supposed to work. I was able to see an example toast notification in the app, and then restarted the app to see the "new update is downloaded..." message. I don't know if I was to expect the "new update" toast to appear on first open. Things seem to be working alright regardless, just will want to confirm exact protocol.

[franknoirot]

Thank you @nadr0 for smoke testing Linux. We are good to merge, just checking on other in-flight changes we might need to coordinate