Skip to content

Next#7292

Merged
JanDeDobbeleer merged 2 commits intomainfrom
next
Feb 9, 2026
Merged

Next#7292
JanDeDobbeleer merged 2 commits intomainfrom
next

Conversation

@JanDeDobbeleer
Copy link
Copy Markdown
Owner

Prerequisites

  • I have read and understood the contributing guide.
  • The commit message follows the conventional commits guidelines.
  • Tests for the changes have been added (for bug fixes / features).
  • Docs have been added/updated (for bug fixes / features).

The original robbyrussell theme in oh-my-zsh shows a yellow ✗ when the
git repository has uncommitted changes. The current oh-my-posh version
shows ✗ only when the last command fails (exit code), which differs
from the original behavior.

This fix:
- Adds fetch_status: true to detect git changes
- Shows ✗ when Working or Staging has changes
- Removes the status segment (exit code) that wasn't part of original
Copilot AI review requested due to automatic review settings February 9, 2026 07:19
Some Nightscout API providers (e.g. T1Pal) return the date field as a
float (e.g. 1770512410938.386). Go's json.Unmarshal cannot decode a
float into an int64 field, causing the segment to silently fail.

Add a custom UnmarshalJSON that accepts both integer and floating-point
JSON numbers, truncating to int64. This preserves the existing Date
int64 type and is fully backward compatible.

Fixes #7284
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the robbyrussell Oh My Posh theme to surface Git working tree status directly in the Git segment, aligning the prompt output more closely with the classic “git:(branch) ✗” behavior.

Changes:

  • Enable fetch_status for the Git segment so .Working/.Staging status fields are available in the template.
  • Extend the Git template to show working changes and display a ✗ indicator when working or staging changes exist.
  • Remove the separate status segment.

@JanDeDobbeleer JanDeDobbeleer enabled auto-merge (rebase) February 9, 2026 07:21
@JanDeDobbeleer JanDeDobbeleer merged commit e875d30 into main Feb 9, 2026
7 checks passed
@JanDeDobbeleer JanDeDobbeleer deleted the next branch February 9, 2026 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants