avalon-vanguard/star-map
2
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Say what formatRadiusKm actually returns

Browse Source 
Its doc comment promised a fallback to Earth radii that the function has never had.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WaySiNst4HhDXBHnMy8p5G
This commit is contained in:
Claude
2026-08-06 03:57:40 +00:00
parent 7b0f32f71a
commit b14ce78c5f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/app/shared/format/quantity.ts
+1 -1
View File
@@ -36,7 +36,7 @@ export function formatPeriod(days: number): string {
return `${(days / 365.25).toFixed(days / 365.25 < 100 ? 1 : 0)} yr`; return `${(days / 365.25).toFixed(days / 365.25 < 100 ? 1 : 0)} yr`;
} }
/** Radius in kilometres, or Earth radii once the number stops being legible in km. */ /** Radius in kilometres. */
export function formatRadiusKm(radiusKm: number): string { export function formatRadiusKm(radiusKm: number): string {
// Grouped on both sides of the decimal threshold: 69,911 km beside a bare 6371 km reads as two // Grouped on both sides of the decimal threshold: 69,911 km beside a bare 6371 km reads as two
// different conventions rather than one. // different conventions rather than one.