docs(ospo): community health rollout v2 — README, agents.md, health files - #41567
Conversation
phil-davis
left a comment
There was a problem hiding this comment.
Not a really important suggestion, but the webui test command was missed.
Otherwise this LGTM, so I am approving.
|
Hi @phil-davis — applied your suggestion, thanks for the approval! The |
Apply suggestion from #41567 (comment)
phil-davis
left a comment
There was a problem hiding this comment.
LGTM now.
It is complaining that the commits are not signed.
Apply suggestion from #41567 (comment) Signed-off-by: David Walter <david.walter@kiteworks.com>
b4ee039 to
f263d31
Compare
DeepDiver1975
left a comment
There was a problem hiding this comment.
Code Review — docs(ospo): community health rollout v2 — README, agents.md, health files
Overview: OSPO-driven documentation update across ownCloud core — rewrites README, adds agents.md, CODE_OF_CONDUCT.md, CONTRIBUTING.md, SECURITY.md, SUPPORT.md.
README.md
The rewrite is comprehensive and well-structured. A few observations:
Positive: The new README is substantially more useful — clear prerequisites, actual build command (make), organized sections for community, contributing, security, and translations. The OSPO/license-migration section is transparent about AGPL-3.0 status and migration prerequisites. ✅
Removed content: The original README's feature list and screenshot were removed in favor of a technical description. Acceptable trade-off for a server-side component repository.
Build badge regression: The original had a Drone CI build status badge. The new version replaces CI badges with License/OSPO/Docker badges. The build status badge is useful for contributors to quickly assess CI health — consider whether it should be re-added, though this is a judgment call.
"Rebase Early/Often" under Contributing workflow: The statement "All commits must be PGP/GPG signed" is accurate per the existing contribution requirements. ✅
agents.md
This is a useful addition for AI coding agents. The architecture key paths, build/test commands, and OSPO policy constraints are all accurate based on the repository structure. The constraint about not introducing copyleft dependencies is particularly important given the Apache 2.0 migration goal. ✅
Health files (CODE_OF_CONDUCT, CONTRIBUTING, SECURITY, SUPPORT)
All four files are redirect stubs pointing to external URLs. This is the standard pattern for organizations centralizing these policies. ✅
SECURITY.md correctly instructs not to open public issues for vulnerabilities and points to security.owncloud.com + YesWeHack. ✅
Summary
| Aspect | Assessment |
|---|---|
| README rewrite | ✅ Comprehensive, accurate, well-organized |
| agents.md | ✅ Useful context for AI tools, accurate architecture paths |
| Health files | ✅ Standard redirect stubs |
| CI badge |
Verdict: Ready to merge.
…iles Introduced by the Kiteworks Open Source Program Office (OSPO) on May 5, 2026. Changes: - README.md: rewritten with OSPO v2 template — license-specific migration guidance, Community & Support section, Contributing workflow, Security section pointing to security.owncloud.com + YesWeHack bug bounty - agents.md: AI agent context file with architecture, build commands, and OSPO Policy Constraints (GitHub Actions, Dependabot, Git Workflow) - CODE_OF_CONDUCT.md: redirect to https://owncloud.com/contribute/code-of-conduct/ - CONTRIBUTING.md: redirect to https://owncloud.com/contribute/ - SECURITY.md: redirect to https://security.owncloud.com + YesWeHack - SUPPORT.md: redirect to https://owncloud.com/contact-us/ + channels OSPO: https://kiteworks.com/opensource Signed-off-by: David Walter <david.walter@kiteworks.com> Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
Apply suggestion from #41567 (comment) Signed-off-by: David Walter <david.walter@kiteworks.com> Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
f263d31 to
e01b238
Compare
|
Note: the text still has a few mentions of “OC10” or “ownCloud 10". Those will become out-of-date. But I suppose that we can edit them when "OC11" is ready and either write "11" or write some words that describe this repo (the “classic” file-sync-and-share server?) |
Summary
This PR is part of the Kiteworks OSPO community health rollout (kiteworks.com/opensource), applied to all ~110 public ownCloud repositories starting May 5, 2026.
Test plan
🤖 Generated with Claude Code as part of the ownCloud OSPO rollout.
Kiteworks OSPO: https://kiteworks.com/opensource