Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@maparent — per your suggestion in the all-hands today: Feel free to add to this where appropriate, provided we’re not modifying any existing logic or functionality. The goal is to keep this PR focused and straightforward. |
maparent
left a comment
There was a problem hiding this comment.
All good , a few optional suggestions.
|
Just an idea: Do we want posthog in dev? Should we have an intermediate function that disables it in dev unless a variable is set? Clearly can be done later, just want to open the discussion about whether that would be useful. |
Probably would be useful, but posthog also has affordances to filter out dev. (Graphs, users, etc) |
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
6ed3fbf to
476e9fa
Compare
Codex:
Added In This PR
Already Had PostHog Before This PR
Summary by CodeRabbit