-
Notifications
You must be signed in to change notification settings - Fork 467
docs: Add developer documentation, AGENTS.md #6448
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
Conversation
- Add missing developer documentation - Refresh CONTRIBUTING.md - Refresh the PR template - Import developer docs and CONTRIBUTING.md to docs.flagsmith.com
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on January 10. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Docker builds report
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6448 +/- ##
=======================================
Coverage 98.06% 98.07%
=======================================
Files 1292 1294 +2
Lines 46509 46537 +28
=======================================
+ Hits 45611 45639 +28
Misses 898 898 ☔ 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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Zaimwa9
left a comment
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.
A first few comments before reading the details
|
@gagantrivedi by the way the hook i was mentioning https://code.claude.com/docs/en/hooks#precompact-input |
Zaimwa9
left a comment
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.
Happy to prove and get going with it
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.
Pull request overview
Copilot reviewed 14 out of 15 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature!Changes
In this PR, we attempt to revamp our developer documentation, and make it accessible for human contributors and LLM-based agents alike.
How did you test this code?
To check changes to docs.flagsmith.com, observed the local Docusaurus build with
make -C docs serve.#6451 was generated by Claude (Opus 4.5) on top of this branch. It wasn't a one-shot, but I did observe Claude consistently following the guidelines, particularly using the Make targets and adhering to the test naming pattern documented in
api/README.md.