Include PR title in lightweight comments#70
Merged
Conversation
When LIGHTWEIGHT_COMMENT is enabled, the Teamwork comments dropped the PR title entirely. Add the title as a bold, linked label after the PR link so brief comments still surface which PR they refer to while staying to a single line, e.g. [PR](url) [**"Fix login"**](url) opened by user
The title is already linked to the PR, so linking the 'PR' label to the same URL was redundant. Keep 'PR' as plain text and link only the title: PR [**"Fix login"**](url) opened by user
Match the bold PR title by also bolding the actor's username, e.g. PR [**"Fix login"**](url) opened by **user**
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
LIGHTWEIGHT_COMMENTkeeps Teamwork comments brief, but it also dropped the PR title — the comment was just[PR](url) opened by user, which gives no hint of which PR a task is referring to.This keeps the comment to a single line but restores the title: the PR title is rendered bold, quoted, and linked to the PR right after the
PRlabel, e.g.The same treatment is applied to all four lightweight events: opened, merged, closed-without-merge, and review-approved. The verbose (non-lightweight) comments are unchanged. README updated to note the title is now included.
Test Task