toolshed/auto-recipes-catalogue-json
2
0
Fork 0
Code Issues 1 Pull Requests Projects Releases Activity

Remove matrix notification, prevent failure when no changes #6

Merged
decentral1se merged 1 commits from :no-matrix-notify into main 2025-07-15 06:12:06 +00:00
Conversation 4 Commits 1 Files Changed 1 +2 -16
Ghost commented 2025-06-28 14:10:02 +00:00
Copy Link

  • Removed the matrix notification on failure. These were going to the room #coopcloud-dev:autonomic.zone but on 2024-12-29 they decided to close that room.

  • Currently, if there are no changes detected, the build will fail in a slightly confusing way since the git commit command fails silently when no changes are detected. I added a message explaining this situation, which also prevents the build from failing.

* Removed the matrix notification on failure. These were going to the room `#coopcloud-dev:autonomic.zone` but on 2024-12-29 they decided to close that room. * Currently, if there are no changes detected, the build will fail in a slightly confusing way since the `git commit` command fails silently when no changes are detected. I added a message explaining this situation, which also prevents the build from failing.
Ghost added 1 commit 2025-06-28 14:10:03 +00:00
Remove matrix notification, prevent failure when no changes d369c9e8fd
decentral1se reviewed 2025-06-28 15:39:46 +00:00
decentral1se left a comment
Owner
Copy Link

Haven't tested this but it looks ✅

Thanks! Let's see if anyone else can review.

Haven't tested this but it looks ✅ Thanks! Let's see if anyone else can review.
decentral1se requested review from 3wordchant 2025-06-28 15:39:52 +00:00
3wordchant commented 2025-07-12 12:57:09 +00:00
Owner
Copy Link

Looks fine to me also. Let's merge and see how it goes

Looks fine to me also. Let's merge and see how it goes
3wordchant approved these changes 2025-07-12 12:57:25 +00:00
Ghost commented 2025-07-15 00:54:55 +00:00
Copy Link

I'm not authorized to merge FYI if you were waiting on me

I'm not authorized to merge FYI if you were waiting on me
👍 1
decentral1se merged commit d66987bfc4 into main 2025-07-15 06:12:06 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
3wordchant
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 Apfelwurm carla cas coopcloud cyrnel decentral1se fauno iexos jade jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband linnealovespie moritz notplants p4u1 pharaohgraphy renovate-bot ripclap simon sixsmith trav val yksflip
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/auto-recipes-catalogue-json#6
No description provided.
Delete Branch ":no-matrix-notify"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?