- 
                Notifications
    
You must be signed in to change notification settings  - Fork 457
 
chore: added-claude-contexts #6155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: chore/claude-context
Are you sure you want to change the base?
Conversation
| 
           The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
  | 
    
          Docker builds report
  | 
    
          Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@                   Coverage Diff                   @@
##             chore/claude-context    #6155   +/-   ##
=======================================================
  Coverage                        ?   98.00%           
=======================================================
  Files                           ?     1278           
  Lines                           ?    44983           
  Branches                        ?        0           
=======================================================
  Hits                            ?    44084           
  Misses                          ?      899           
  Partials                        ?        0           ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we're adding this, the frontend claude context should point to this as the source of truth. This makes me think this should all be 1 PR with a claude.md in api / frontend.
The backend and frontend contexts maybe shouldn't be in the same folder since I imagine it could cause increased token usage. However, I think the FE should point to files like this where applicable.
| 
           matthewelwell commented:  | 
    
| 
           khvn26 commented:  | 
    
| 
           Test new thread  | 
    
| 
           Zaimwa9 commented:  | 
    
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature!Changes
Related to #6153
How did you test this code?