toolshed/auto-recipes-catalogue-json
1
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 (deleted):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
Copy Source

  • 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
Copy Source

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
Copy Source

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
Copy Source

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
decentral1se
3wordchant
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) Apfelwurm BornDeleuze Brooke carla cas (Cassowary) coopcloud cyrnel decentral1se (d1) dede devydave fauno (fauno) iexos jade (Jade Ambrose) jjsfunhouse jmakdah2 (Jackie Makdah) joe-irving (Joe Irving) kawaiipunk (KawaiiPunk) knoflook kolaente lambdabundesverband linnealovespie (April) moosemower moritz notplants oxaliq (sorrel) p4u1 pharaohgraphy (Andrew 🐦‍🔥❤️‍🔥✴️) renovate-bot (Comrade Renovate Bot) ripclap simon sixsmith (Sixsmith) stevensting trav (Trav Fryer) val (val (he/him)) yksflip
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/auto-recipes-catalogue-json#6