Skip to content

switch eyre to anyhow and misc cleanup - #8311

Open
aecsocket wants to merge 8 commits into
mainfrom
boris/nix-flake
Open

aecsocket wants to merge 8 commits into
mainfrom
boris/nix-flake

Conversation

@aecsocket

Copy link
Copy Markdown
Contributor
  • add flake.nix devenv
  • remove maxmind crate

@github-actions

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@aecsocket
aecsocket marked this pull request as ready for review November 13, 2026 16:12
@mergify

mergify Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Queue requested — the merge queue status continues in this comment ↓.

@greptile-apps

greptile-apps Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable defects were found.

Summary

Replaces eyre with anyhow across Labrinth and shared Rust packages.

  • Labrinth and its Rust support crates now use Anyhow for errors.
  • The optional MaxMind country-lookup crate is removed.
  • A Nix development shell provides Labrinth's Rust and service tools.

No actionable issues were found. Builds and tests were not run.

Reviews (1) · Last reviewed commit: "switch eyre to anyhow" · Reviewed by Greptile

greptile-apps[bot]
greptile-apps Bot previously approved these changes Nov 13, 2026
@greptile-apps
greptile-apps Bot dismissed their stale review November 13, 2026 16:28

Dismissed because a newer commit was pushed; Greptile will re-review the current head.

@aecsocket

Copy link
Copy Markdown
Contributor Author

@Mergifyio queue

@mergify

mergify Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

🚫 Left the queue · conflict with base branch

Less than a minute in the queue, with no time running CI.

So far

Reason

The pull request conflicts with main

Conflicting file: apps/labrinth/src/routes/internal/session.rs

Hint

You should update or rebase your pull request. A pull request queued by a rule comes back to the queue once updated, as long as the rule still matches.

If it was queued with a command, post a new @mergifyio queue comment to requeue it.

Tick the box to put this pull request back in the merge queue (same as @mergifyio queue).

  • Requeue this pull request

Times in UTC · dashboard · how the merge queue works

@mergify mergify Bot removed the dequeued label Nov 13, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants