Skip to content

doc: change filesize unit tip to warning for clarity#2145

Closed
CAESIUS-TIM wants to merge 2 commits intonushell:mainfrom
CAESIUS-TIM:patch-2
Closed

doc: change filesize unit tip to warning for clarity#2145
CAESIUS-TIM wants to merge 2 commits intonushell:mainfrom
CAESIUS-TIM:patch-2

Conversation

@CAESIUS-TIM
Copy link
Copy Markdown
Contributor

Change from tip to warning to highlight potential pitfall: case does not distinguish between bits and bytes.

Change from tip to warning to highlight potential pitfall:
case does not distinguish between bits and bytes.
@Juhan280
Copy link
Copy Markdown

Juhan280 commented Apr 1, 2026

Can you explain what you are talking about

@CAESIUS-TIM
Copy link
Copy Markdown
Contributor Author

In general, an uppercase B stands for byte, while a lowercase b represents bit.
Therefore, if file size unit is case-insensitive, this comment had better be marked as a warning to avoid potential pitfalls

@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Apr 1, 2026

I think it's clear enough how it's already written. It says they're insensitive and equivalent.

@fdncred fdncred closed this Apr 1, 2026
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.

3 participants