Skip to content

Make root PHILOSOPHY.md a symlink to the docs philosophy page#13954

Open
yiyixuxu wants to merge 3 commits into
mainfrom
symlink-philosophy-doc
Open

Make root PHILOSOPHY.md a symlink to the docs philosophy page#13954
yiyixuxu wants to merge 3 commits into
mainfrom
symlink-philosophy-doc

Conversation

@yiyixuxu

Copy link
Copy Markdown
Collaborator

No description provided.

The root PHILOSOPHY.md had drifted into a stale duplicate of
docs/source/en/conceptual/philosophy.md (which was rewritten in #13808 to
cover the single-file model policy and Modular Diffusers). Replace it with a
symlink to the docs file, mirroring CONTRIBUTING.md -> contribution.md, so
there's a single source of truth.

A symlink renders as a path stub (not content) on GitHub, so repoint the
links that pointed at blob/main/PHILOSOPHY.md to the rendered docs site:
the PR template and the English contribution guide.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions github-actions Bot added documentation Improvements or additions to documentation CI size/M PR with diff < 200 LOC labels Jun 14, 2026
yiyixuxu and others added 2 commits June 14, 2026 21:01
CONTRIBUTING.md is a symlink, so blob/main/CONTRIBUTING.md renders as a path
stub on GitHub rather than the guide. Repoint the PR template and README
links to the rendered docs page, matching the philosophy/AI-agents links.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The stale-bot comment and the ko/zh contribution guides still linked to
blob/main/{CONTRIBUTING,PHILOSOPHY}.md, which render as path stubs because
those root files are symlinks. Point them at the rendered docs pages
(localized /ko/ and /zh/ for the translations).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Comment thread PHILOSOPHY.md
@@ -1,110 +0,0 @@
<!--Copyright 2025 The HuggingFace Team. All rights reserved.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I realized that this file is still outdated after I updated the philosophy doc in #13808
so made it a symlink (follow same pattern as CONTRIBUTING.md

- [ ] Did you use an AI agent (Claude Code, Codex, Cursor, etc.) to help with this PR? If so:
- [ ] Did you point it at the project conventions in [`.ai/`](https://github.com/huggingface/diffusers/tree/main/.ai) (e.g. via `make claude` / `make codex`)? See [Coding with AI agents](https://huggingface.co/docs/diffusers/main/en/conceptual/contribution#coding-with-ai-agents).
- [ ] Did you self-review the diff against [`.ai/review-rules.md`](https://github.com/huggingface/diffusers/blob/main/.ai/review-rules.md)?
- [ ] Did you read the [contributor guideline](https://github.com/huggingface/diffusers/blob/main/CONTRIBUTING.md)?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

pointed all the links to rendered doc page directly
the links correctly open like https://github.com/huggingface/diffusers/blob/main/CONTRIBUTING.md

@yiyixuxu yiyixuxu requested review from sayakpaul and stevhliu June 15, 2026 02:25
Comment thread PHILOSOPHY.md

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.

Suggested change
[docs/source/en/conceptual/philosophy.md](./docs/source/en/conceptual/philosophy.md)

This way, it renders as a path?

@github-actions

Copy link
Copy Markdown
Contributor

Hi @yiyixuxu, thanks for the PR! It does not appear to link an issue it fixes. If this PR addresses an existing issue, please add a closing keyword (e.g. Fixes #1234) to the PR description so the issue is linked. See the contribution guide for more details. If this PR intentionally does not fix a tracked issue, a maintainer can add the no-issue-needed label to silence this reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI documentation Improvements or additions to documentation size/M PR with diff < 200 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants