Skip to content

Make CLAUDE.md just include @AGENTS.md#412

Merged
MichaelHatherly merged 5 commits intoPumasAI:mainfrom
visr:symlink
May 1, 2026
Merged

Make CLAUDE.md just include @AGENTS.md#412
MichaelHatherly merged 5 commits intoPumasAI:mainfrom
visr:symlink

Conversation

@visr
Copy link
Copy Markdown
Contributor

@visr visr commented Apr 30, 2026

Same like JuliaLang/julia#60722.

I ran into this installing Quarto from conda-forge with Pixi on Windows (unrelated to Julia).

❯ pixi add quarto
Error:   × failed to link quarto-1.9.37-h0e8a320_0.conda
  ├─▶ failed to link 'Library/share/quarto/extension-subtrees/julia-engine/CLAUDE.md'
  ├─▶ could not open source file
  ╰─▶ The system cannot find the file specified. (os error 2)

I can also raise this at https://github.com/conda-forge/quarto-feedstock, perhaps a workaround is needed there.

Copy link
Copy Markdown
Collaborator

@MichaelHatherly MichaelHatherly left a comment

Choose a reason for hiding this comment

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

Thanks, we can change from a symlink to explicit @ ref, that's fine.

Comment thread CLAUDE.md Outdated
Comment on lines 1 to 8
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Would prefer not adding the explicit comment about @ syntax.

Suggested change

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done in 19ef636, since your suggested change didn't seem to remove the whole comment.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.18%. Comparing base (49676a2) to head (b7ba3d6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
- Coverage   81.49%   81.18%   -0.31%     
==========================================
  Files          73       73              
  Lines        2923     2923              
==========================================
- Hits         2382     2373       -9     
- Misses        541      550       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Co-authored-by: Copilot <[email protected]>
Comment thread CHANGELOG.md

### Changed

- Remove `CLAUDE.md` symlink to avoid Quarto packaging issues [#412]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Will need the issue URL listed at the end of the file as well.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh right, that makes sense. Done.

Co-authored-by: Copilot <[email protected]>
@MichaelHatherly MichaelHatherly merged commit 37a59c0 into PumasAI:main May 1, 2026
11 of 12 checks passed
@visr visr deleted the symlink branch May 1, 2026 10:11
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.

2 participants