Repository navigation
fix(root): remove the root CODE_OF_CONDUCT.md; the fork keeps it under .github/ - #175
Conversation
…r .github/ The upstream merges left a second copy of the code of conduct at the repository root, byte-identical to .github/CODE_OF_CONDUCT.md. The root shape allowlist in test/unit/test_install_pins.jl forbids it -- health files live under .github/ -- so the Julia suite on main fails at "the root keeps its shape": root entries outside the allowlist: CODE_OF_CONDUCT.md. MEASURED on main's da9754f CI run. The .github/ copy is the one GitHub and the fork's doctrine use; the root copy is dropped. Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 59 minutes. View limit details
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📜 Recent review details
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Description check |  |
The description clearly explains the file deletion, the reason for the change, the affected test, and the reported validation command. It omits several template sections, including the change type, ch… |
| Title check | |
The title is concise, specific, and accurately identifies the main change: removing the duplicate root CODE_OF_CONDUCT.md while retaining the copy under .github/. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
|
1 participant
What this change does
Removes
CODE_OF_CONDUCT.mdfrom the repository root. The byte-identical copy at.github/CODE_OF_CONDUCT.mdstays.Why
The upstream merges (#166-#174) left a second copy of the code of conduct at the root. The root-shape allowlist in
test/unit/test_install_pins.jl("the root keeps its shape", from the RSR root-cleanup doctrine) forbids root entries outside the allowlist — health files live under.github/— so the Julia suite on main fails:MEASURED on main's da9754f CI run. GitHub reads the code of conduct from
.github/; the root copy is dropped, not merged.How to run its tests
julia --project=. -e 'using Test; include("test/unit/test_install_pins.jl")'— or the full suite; the "the root keeps its shape" testset is the affected one.Versions tested against
Not applicable — a file deletion. Julia 1.13.1, R 4.5.3 (conda-forge) for the suite context.
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe vibe@mistral.ai