matrix-org / matrix-org/matrix-hookshot
GitHub notification of PR merges is extremely verbose
Open
Nobody has claimed this yet.
GitHub
S-Minor
T-Defect
- Dominant language
- TypeScript
- Stars
- 450
- Forks
- 95
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 23
Description
this is a PR merge, it is very verbose:
could be a case to use <details> if the OP needs to be sent at all? This one also includes html comments <!-- --> literally.
on github: https://github.com/matrix-org/matrix-react-sdk/pull/5887#pullrequestreview-858005967
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the GitHub pull-request merge notification handling and compare its output with the linked GitHub example. Done means merge notifications are no longer unnecessarily verbose, optional content can be collapsed or omitted, and HTML comments are not shown literally.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github, typescript
- Domain
- api, backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100