-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
File based settings #1679
File based settings #1679
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
+ change the identifier back since it really doesn't seem to affect app signing
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1679 +/- ##
==========================================
+ Coverage 84.64% 87.49% +2.85%
==========================================
Files 44 46 +2
Lines 18185 18946 +761
==========================================
+ Hits 15392 16577 +1185
+ Misses 2793 2369 -424
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Resurrecting my effort for file-based settings because it has some really valuable logic for falling back to defaults if invalid (or, critically, deprecated) settings values are found in the user's configs, which will allow me to safely rename the mouse control setting to "Zoo" from "KittyCAD". Likely still Tauri test fixes and other cleanup to do since we've been cranking so much the past few weeks