v0.3.2 backportapalooza #170
Conversation
* Refactor: Remove unneccesary CLI branching * Refactor: Refactor configuration load function * Feat: Add '--config' flag * Refactor: Reduce passing of args and configuration * Fix: Fix cli parsing of page width * Clean: use 'display()' instead of debug print for path * Clean: use Default for Keybindings * Refactor: Use Keybindings in tests * Doc: Fix 'page width' to 'page_width' * Refactor: use interpolation style * Refactor: Remove inline on Keybindings::new * Refactor: Extract Keybindings Default implementation * Refactor: Destructure Config and Args in opts
* Make correct location of `inlyne.toml` file clearer * Fix missing backtick
* Cleanup existing custom debug impls * Add custom debug impls for `Element::Image(_)` * Add a snapshot test for complex image elements
* The watcher should attempt to re-register on name changes * Re-register watcher after longer delays * Add a reminder for fixing watching after file changes * Move file watcher into its own file * Make file watcher testable * Watcher follows file changes * Test the file watcher * Final test tweaks * Swap back to the old delays * Final final test tweaks * Use a slightly longer delay * Shorter poll interval for file watcher * Follow through with rename
* Rework panic happy file reloading * Tweak logs * The watcher should attempt to re-register on name changes * Re-register watcher after longer delays
* Add failing test * Use extended syntax definitions * Add failing test * Allow for comma as an info string delimiter
* Add failing tests * Handle nesting of lists * Cleanup list prefix handling
This solves an issue with the NixOS build where fonts are required at runtime for inlyne to start. With the `fontconfig` feature flag inlyne will use font-config to determine a font. Partially resolves Inlyne-Project#164
* Use `human-panic` for a custom panic hook * Indicate that the version is a "git release"
|
Whelp... this will have to be on hold until I'm no longer sick 🤒 |
CosmicHorrorDev
force-pushed
the
v0.3.2-backportapalooza
branch
from
85fb74d
to
a26509e
Compare
|
@AlphaKeks whenever you have time (no rush at all) could you check to see if this branch still fixes that panic you were experiencing that you recently fixed on |
|
I can confirm that, with What's weird is that I'm honestly confused, but whatever it is, it's not inlyne's issue. |
|
Well happy to hear that things are working at least! |
This bundles together all the relevant bug fixes, documentation changes, and refactors/tests that make sense for a patch release. Notably it includes all of these PRs
inlyne.tomlfile clearer #122fontdbfeature flag #169Cargo.toml#171human-panicfor a custom panic hook #172TODO: This has that zbus issue that causes color theme detection to fail on Linux. Check to see if that's fixed when updating depsLooks like the existing issue should be fixed and I'm now hitting https://bugzilla.redhat.com/show_bug.cgi?id=2239248 🙃TODO: Need to follow through all the regular release steps. Bump version, update deps, draft up a more coherent description of the changes, etc.
$ cargo +nightly udepsrust-versioninCargo.toml$ cargo msrv --min 1.60 -- cargo checkversioninCargo.tomlcargo checkto propagate changes toCargo.lock$ cargo publish$ git tag v{VERSION}(make sure the branch you are on is up to date)$ git push upstream/origin v{VERSION}