matrix-org / matrix-org/matrix-hookshot
Strip HTML comments from gh issue/pr bodies in gh notifications
- Dominant language
- TypeScript
- Stars
- 450
- Forks
- 95
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 23
Description
issue/pr templates often use lengthy comments like this, but there is really no need to spam notifications with them i think.
they could also be hidden in element by allowing html comments in html bodies, but that seems to make little sense
Contributor guide
Research direction
Start by locating the GitHub issue and pull-request notification body handling in the TypeScript code, then inspect how HTML content is currently passed through. The work is done when HTML comments from GitHub issue and PR bodies no longer appear in Matrix notifications, with the relevant existing behavior still covered by tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100