Remove notify_failures from test-all-warehouses.yml

[devin-ai-integration]

Summary

Removes the notify_failures job from test-all-warehouses.yml. This job called notify_slack.yml to send Slack notifications on test failures, but it is not needed and currently creates irrelevant failures.

Review & Testing Checklist for Human

• Confirm no other workflows or processes depend on the notify_failures job in this workflow
• Check if .github/workflows/notify_slack.yml is still referenced elsewhere — if not, consider removing it in a follow-up

Notes

• Requested by: @haritamar
• Link to Devin run
• A matching PR is being created in dbt-data-reliability for the same change.

Summary by CodeRabbit

• Chores
  • Removed Slack notification workflow for test completions.

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[github-actions]

👋 @devin-ai-integration[bot]
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in this pull request.

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request removes a Slack notification workflow invocation from the test-all-warehouses workflow. The notify_failures step, which sent notifications to Slack upon test completion, has been deleted entirely from the workflow configuration.

Changes

Cohort / File(s)	Summary
Workflow Configuration .github/workflows/test-all-warehouses.yml	Removed Slack notification step (notify_failures) that was triggered after test matrix completion, including workflow reference and input parameters (result, run_id, workflow_name).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A whisker of change, a notification gone,
No more Slack alerts when tests are done,
The pipeline runs clean, so swift and light,
Fewer distractions in the workflow night! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and specifically describes the main change: removing the notify_failures step from the test-all-warehouses.yml workflow file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch devin/1770668789-remove-notify-failures

---

No actionable comments were generated in the recent review. 🎉

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}