Skip to content

Conversation

@ralfhandl
Copy link
Contributor

Use title case for headings.

@ralfhandl ralfhandl added this to the Release 1.1 milestone Nov 4, 2025
@ralfhandl ralfhandl requested a review from a team as a code owner November 4, 2025 18:13
Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

ah sorry for missing this convention

@mikeschinkel
Copy link

I am curious what the motivation was for this? Not that I (necessarily) disagree with it, but would like to know the rationale?

I have held different opinions on the use of title case over the years — both preferring at times and at other times preferring against it — but seeing this I realized that I did not have a strong basis for my opinion so I asked ChatGPT which gave me much food for thought:

Given that, I'd be curious to know the reasons for title case based on ChatGPT's moderate recommendation of sentence case?

@lornajane
Copy link
Contributor

Consistency is king, and we're not a modern project

@handrews
Copy link
Member

handrews commented Nov 4, 2025

@lornajane arguably, inconsistent capitalization would make us a postmodern project 😜

Copy link
Contributor

@lornajane lornajane left a comment

Choose a reason for hiding this comment

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

Good catch

@lornajane lornajane merged commit 2491dc6 into OAI:main Nov 4, 2025
3 checks passed
@ralfhandl ralfhandl deleted the heading-style branch November 4, 2025 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants