Skip to content

Fix grammar in language/types - #5746

Merged
kamil-tekiela merged 3 commits into
php:masterfrom
kamil-tekiela:Fix-grammar-in-language/types
Aug 7, 2026
Merged

Fix grammar in language/types#5746
kamil-tekiela merged 3 commits into
php:masterfrom
kamil-tekiela:Fix-grammar-in-language/types

Conversation

@kamil-tekiela

Copy link
Copy Markdown
Member

The sentence in iterable.xml seems wrong or maybe I cannot understand it, but I don't know how to fix it.

@kamil-tekiela
kamil-tekiela force-pushed the Fix-grammar-in-language/types branch from ba11f6c to 2063895 Compare August 7, 2026 12:21
@kamil-tekiela
kamil-tekiela force-pushed the Fix-grammar-in-language/types branch from 2063895 to c14dfe5 Compare August 7, 2026 12:26

@jordikroon jordikroon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some minor (non blocking) nits. Feel free to choose whatever you feel right with.

Comment thread language/types/string.xml Outdated
<note>
<simpara>
On 32-bit builds, a <type>string</type> can be as large as up to 2GB
On 32-bit builds, a <type>string</type> can be as large as 2GB

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How about this:

On 32-bit builds, a <type>string</type> can be up to 2GB (2147483647 bytes)..

Comment thread language/types/mixed.xml
<simpara>
<type>mixed</type> is, in type theory parlance, the top type,
meaning every other type is a subtype of it.
</simpara>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

While I agree with the changes. This phrase is that advanced that I don't believe the majority of the people that read the documentation will understand. Read: Non-native speakers.

My suggestion:
<type>mixed</type> is a special type that accepts values of any other type.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That's already on the page a couple of lines above it. I agree, it's difficult to understand, but not sure that dumbing it down is a good idea. It says exactly what it was supposed to say for people who are interested in type theory. An average reader will stop on the first paragraph.

@kamil-tekiela
kamil-tekiela merged commit 7132d88 into php:master Aug 7, 2026
2 checks passed
@kamil-tekiela
kamil-tekiela deleted the Fix-grammar-in-language/types branch August 9, 2026 11:34
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.

2 participants