Put a space between the commodity symbol and the quantity

Open Beginner friendly

#73 opened on Aug 25, 2026 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 2/5
Estimated time: 1-3 hours
Newbie friendliness: 88/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Active
Tech stack: fsharp
Domain: backend

Research direction

Start in Amount.ToString, using AmountStyle.CommoditySpaced and the referenced BalanceReport.fs lines to trace the formatting. Update the expectations in Tabularius.Tests/Data/AmountTests.fs, and use Tabularius.Tests/Interop/BalanceReportTests.fs to confirm the spaced style is preserved. Done means a true value produces one space between the commodity symbol and quantity.

Written by the indexing model from the issue text.

Description

enhancement good first issue help wanted

Problem

The AmountStyle type has a CommoditySpaced field. Amount.ToString does not read this field. The method puts the commodity symbol directly before the quantity or directly after the quantity. The method never puts a space between these two parts.

Do these steps:

Read AmountStyle.CommoditySpaced in Amount.ToString.
Put one space between the commodity symbol and the quantity if the value of the field is true.
Correct the tests in Tabularius.Tests/Data/AmountTests.fs. These tests hold the current behavior.

Why this task is necessary

hledger reports the space in the amount style. The interop layer keeps this data, and Tabularius.Tests/Interop/BalanceReportTests.fs shows the value. The program then removes the space when it shows the amount. The user sees 100USD in place of 100 USD.

Code

https://github.com/ForNeVeR/tabularius/blob/c72791e9b177ccc5bf44c4cbe66048605f2fe3c6/Tabularius.Data/BalanceReport.fs#L35-L41

Dominant language: F#
Stars: 1
Forks: 0
Avg merge: 12h 14m
Merged PRs (30d): 10

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

[Bug]: Matrix progress drafts fail with "Matrix runtime not initialized" during tool activity Open

bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

openclaw/openclaw#155298 · 2 comments · 1 reaction · Opened Sep 22, 2026
[Bug] NULL pointer dereference in llm_running_callback (NGZA-AX-LLM #3) Open

Difficulty 1/5 Under an hour Newbie friendliness 90/100

AXERA-TECH/ax-llm#77 · Opened Sep 22, 2026
resume() sends a second HTTP response (400) after a successful resume Open

Difficulty 1/5 Under an hour Newbie friendliness 90/100

games-on-whales/wolf#509 · Opened Sep 21, 2026
MCP client has no httpx timeout, so the event stream fails after 5 seconds Open

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

Azure/azure-functions-agents-runtime#228 · Opened Sep 21, 2026
feat(mcp): friendlier error when `run_cloud_sync` is called on a disabled connection Open

enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 72/100

airbytehq/PyAirbyte#1205 · Opened Sep 21, 2026

More Backend & API Design issues