mattermost / mattermost/mattermost-plugin-github

Have "PR Merged" subscriptions support filtering labels, for an attached issue

Open
#741 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Help Wanted Up For Grabs
Dominant language
Go
Stars
179
Forks
180
Avg merge
1d 16h
Merged PRs (30d)
13

Description

When a PR is merged, the issue it's attached to contains context for the purpose of this PR. It would be great to have a message like:

[PR x] was merged, with an [issue y] attached with [label z]

The task here is to make pulls and pulls_merged support searching through issues linked to the PR. If there is an error fetching the issue or labels, we should ignore and move on with checking other subscriptions. Note that only one post should be made for a channel that has multiple subscriptions.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No files or tests are named. Start by tracing the subscription handling for pulls and pulls_merged, then inspect how linked issues and labels are fetched. Done means those subscriptions can filter by labels on attached issues, fetching errors do not stop other subscriptions, and a channel receives only one post when subscriptions overlap.

Written by the indexing model from the issue text.

Assessment

Tech stack
github, go
Domain
api, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.