Skip to content

Correct class name formatting in locking.md#8316

Merged
dereuromark merged 2 commits into
cakephp:5.nextfrom
PhantomWatson:patch-2
Jun 30, 2026
Merged

Correct class name formatting in locking.md#8316
dereuromark merged 2 commits into
cakephp:5.nextfrom
PhantomWatson:patch-2

Conversation

@PhantomWatson

Copy link
Copy Markdown
Contributor
`class` Cake\Lock\**Lock**

was making the lock page's header show up as
image

Copilot AI review requested due to automatic review settings June 29, 2026 22:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Locking documentation page’s “class” header line to avoid malformed rendering caused by the previous mixed Markdown formatting.

Changes:

  • Replace the class header line in locking.md to correct how the class name is displayed.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/en/core-libraries/locking.md Outdated
@dereuromark

Copy link
Copy Markdown
Member

The render glitch comes from the single backslash: \L is treated as a Markdown escape, which eats the L and breaks the header. Dropping the bold isn't needed — the real fix is to escape the namespace separators and keep the short class name bold, matching every other page (e.g. caching.md:8Cake\\Cache\\**Cache**, views.md:8Cake\\View\\**View**).

`class` Cake\\Lock\\**Lock**

That keeps locking.md consistent with the rest of the docs.

A single backslash before `L` was treated as a Markdown escape, eating the
`L` and breaking the header. Escape the namespace separators with double
backslashes and keep the short class name bold, matching every other page
(e.g. caching.md, views.md).
@dereuromark dereuromark merged commit 22f9822 into cakephp:5.next Jun 30, 2026
6 checks passed
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