Skip to content

prisma-taurus: drop the Slack "ready for review" notification step - #2020

Merged
RafaelGuevaraCA merged 1 commit into
masterfrom
remove-slack-step-prisma-taurus
Sep 2, 2026
Merged

prisma-taurus: drop the Slack "ready for review" notification step#2020
RafaelGuevaraCA merged 1 commit into
masterfrom
remove-slack-step-prisma-taurus

Conversation

@RafaelGuevaraCA

Copy link
Copy Markdown
Collaborator

Summary

  • The skill no longer posts a "ready for review" message to Slack after opening a PR.
  • Removes step 17 (the chat.postMessage call and its message-format spec) and the optional SLACK_BOT_TOKEN / SLACK_SPARTA_CHANNEL_ID credentials block.
  • Renumbers the subsequent steps and updates cross-references accordingly.

Test plan

  • Docs-only change (SKILL.md) — no code, no tests to run.

The skill no longer posts to Slack. Removes step 17 (the best-effort
chat.postMessage post and its message-format spec), the optional
SLACK_BOT_TOKEN / SLACK_SPARTA_CHANNEL_ID credentials block, and the
Slack line from the final-summary template.

Renumbers the following steps (18 -> 17, 19 -> 18) and updates the
cross-references in the overview, the scanner-appeasement gate, and the
republish reference section.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.19%. Comparing base (7a06796) to head (c4e38af).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2020   +/-   ##
=======================================
  Coverage   88.19%   88.19%           
=======================================
  Files          75       75           
  Lines       21162    21162           
=======================================
  Hits        18661    18661           
  Misses       2501     2501           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@RafaelGuevaraCA
RafaelGuevaraCA merged commit 806cb89 into master Sep 2, 2026
3 checks passed
@RafaelGuevaraCA
RafaelGuevaraCA deleted the remove-slack-step-prisma-taurus branch September 2, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant