Skip to content

Introduce gitlabmergerequestcomment notification provider#1231

Merged
matheuscscp merged 2 commits intomainfrom
gl-mr-comment
Feb 6, 2026
Merged

Introduce gitlabmergerequestcomment notification provider#1231
matheuscscp merged 2 commits intomainfrom
gl-mr-comment

Conversation

@matheuscscp
Copy link
Member

@matheuscscp matheuscscp commented Feb 5, 2026

Closes: #1073

Also, refactor too many providers checking commit status. Should be done in a single place.

Screenshot from 2026-02-06 11-11-45 Screenshot from 2026-02-06 11-10-26

@matheuscscp matheuscscp added enhancement New feature or request area/alerting Alerting related issues and PRs labels Feb 5, 2026
Base automatically changed from gh-pr-comment to main February 6, 2026 11:03
Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>
Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks @matheuscscp 🥇

PS. Most needed refactoring in Flux GG!

@matheuscscp matheuscscp merged commit 0e2d580 into main Feb 6, 2026
5 checks passed
@matheuscscp matheuscscp deleted the gl-mr-comment branch February 6, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/alerting Alerting related issues and PRs enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Provider types githubpullrequestcomment and gitlabmergerequestcomment

2 participants