chore: updates the PR template coding conventions message #31027
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
Updated the PR template to include a prominent, GitHub callout-style alert reminding contributors to stay updated with PostHog coding conventions, while redundant duplicate alerts have been removed.
- Modified file:
.github/pull_request_template.mdto add an important alert. - Implemented GitHub callout syntax linking to PostHog coding conventions.
- Removed duplicate alert for cleaner, consolidated guidance.
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile
rodrigoi
changed the title
patricio-posthog
approved these changes
Apr 9, 2025
rodrigoi
deleted the
chore/adds-important-alert-for-code-review-documentation
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Important
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
Problem
A small change to the PR template to use an alert to link to the Coding Convetions documentation.
Changes
Does this work well for both Cloud and self-hosted?
How did you test this code?