Skip to content

Conversation

@prashant-gurung899
Copy link
Contributor

@prashant-gurung899 prashant-gurung899 commented Jul 18, 2025

Description

This PR adds a new pipeline to enable sending CI notifications to a Matrix channel.

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • 🤖

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added

@prashant-gurung899 prashant-gurung899 self-assigned this Jul 18, 2025
@prashant-gurung899 prashant-gurung899 force-pushed the matrix-notifications branch 8 times, most recently from f679620 to 627c418 Compare July 23, 2025 04:29
@prashant-gurung899 prashant-gurung899 force-pushed the matrix-notifications branch 15 times, most recently from a6f8d3a to de14aa4 Compare August 6, 2025 09:13
@prashant-gurung899 prashant-gurung899 force-pushed the matrix-notifications branch 4 times, most recently from 5de8c64 to c4a51fe Compare August 12, 2025 04:25
@prashant-gurung899 prashant-gurung899 force-pushed the matrix-notifications branch 2 times, most recently from 8561cf7 to 7d71af4 Compare August 13, 2025 06:18
Signed-off-by: prashant-gurung899 <prasantgrg777@gmail.com>
channel = config["rocketchat"]["channel_cron"]

return {
def notifyMatrix(ctx):

Choose a reason for hiding this comment

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

Suggested change
def notifyMatrix(ctx):
def notify(ctx):

Copy link
Contributor

Choose a reason for hiding this comment

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

let's keep notifyMatrix because it is the same in web, or we have to change it in both repos

@prashant-gurung899 prashant-gurung899 merged commit 9462ad5 into main Aug 13, 2025
75 checks passed
@prashant-gurung899 prashant-gurung899 deleted the matrix-notifications branch August 13, 2025 11:19
@openclouders openclouders mentioned this pull request Aug 13, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants