Skip to content

fix: archived versions get unknown status - #8266

Merged
mergify[bot] merged 2 commits into
mainfrom
boris/archived-versions
Nov 10, 2026
Merged

mergify[bot] merged 2 commits into
mainfrom
boris/archived-versions

Conversation

@aecsocket

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Nov 9, 2026

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

@greptile-apps

greptile-apps Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] This fix appears safe to merge.

Summary

Fixes VersionStatus::from_string so "archived" returns VersionStatus::Archived instead of VersionStatus::Unknown.

  • Archived versions keep their status when Labrinth reads them.

Reviews (1) · Last reviewed commit: "fix: archived versions get unknown statu..." · Reviewed by Greptile

@mergify

mergify Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

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

@mergify

mergify Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

✅ Merged into main as bea7c9d at 21:56 UTC

1 min in the queue, with no time running CI.

So far

2 merge conditions met
  • github-review-approved · required by GitHub branch protection
  • any of · required by GitHub branch protection:
    • check-skipped = Lint and Test
    • check-neutral = Lint and Test
    • check-success = Lint and Test

Times in UTC · dashboard · how the merge queue works

@mergify mergify Bot added the queued label Nov 10, 2026
@mergify
mergify Bot merged commit bea7c9d into main Nov 10, 2026
11 checks passed
@mergify
mergify Bot deleted the boris/archived-versions branch November 10, 2026 21:56
@mergify mergify Bot removed the queued label Nov 10, 2026
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