Skip to content

Removes CLAUDE.md - #23784

Open
NickSdot wants to merge 1 commit into
php:masterfrom
NickSdot:chore/claude
Open

NickSdot wants to merge 1 commit into
php:masterfrom
NickSdot:chore/claude

Conversation

@NickSdot

@NickSdot NickSdot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

No longer needed since yesterday:

cc @iluuu1994, @LamentXU123


Edit: reminder that this has to target 8.6 after the branch cut.

@bukka

bukka commented Sep 20, 2026

Copy link
Copy Markdown
Member

I would leave it for a bit longer as not everyone immediately updates and the main purpose of this is for their claude code to read it when they search for security issues...

@LamentXU123

Copy link
Copy Markdown
Member

Yes. And we are yet to see whether that new config is on by default

@NickSdot

Copy link
Copy Markdown
Contributor Author

Yes. And we are yet to see whether that new config is on by default

Fromt the link above.

image

@iliaal

iliaal commented Sep 20, 2026

Copy link
Copy Markdown
Member

I think this really should wait couple of weeks before merging, to let the upgrade cycle to a version of Claude code that adds this feature to go through.

@jorgsowa jorgsowa changed the title Removes Claude marketing Removes CLAUDE.md Sep 21, 2026

@jorgsowa jorgsowa 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.

Claude Code updates itself automatically. IMO, no need to wait. Thanks!

FYI, I renamed the title, because the term claude marketing is not very relevant.

@bukka

bukka commented Sep 22, 2026

Copy link
Copy Markdown
Member

Claude Code updates itself automatically.

This depends on installation / configuration and is not always the case.

As I said before this is not done to make user experience better. We use this purely to reduce number of bogus security reports. This is just for security team to decide though.

@bukka

bukka commented Sep 22, 2026

Copy link
Copy Markdown
Member

It's questionable if this is helping much but if it can possibly prevent few more such reports, it's still good to keep it for a little bit longer.

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.

6 participants