Skip to content

Comments

docs: precede code snippet with standard sentence formatting#2673

Merged
zimeg merged 1 commit intoai-appsfrom
zimeg-docs-ai-colon
Oct 6, 2025
Merged

docs: precede code snippet with standard sentence formatting#2673
zimeg merged 1 commit intoai-appsfrom
zimeg-docs-ai-colon

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Oct 6, 2025

Summary

This PR improves the formatting of a code snippet following a link.

Requirements

@zimeg zimeg requested review from haleychaas and srtaalej October 6, 2025 19:11
@zimeg zimeg self-assigned this Oct 6, 2025
@zimeg zimeg added docs M-T: Documentation work only semver:patch labels Oct 6, 2025
Copy link
Member Author

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

📝 Notes for the wonderful reviewers!

app.event('reaction_added', async ({ event, say }) => {
if (event.reaction === 'calendar') {
await say({
blocks: [{
Copy link
Member Author

Choose a reason for hiding this comment

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

🪬 note: Refactoring this to make copy and pasting a bit easier!

Comment on lines +52 to +54
- [`chat.startStream`](/reference/methods/chat.startstream)
- [`chat.appendStream`](/reference/methods/chat.appendstream)
- [`chat.stopStream`](/reference/methods/chat.stopstream)
Copy link
Member Author

Choose a reason for hiding this comment

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

👁️‍🗨️ note: Using dashes instead of asterisk to avoid possible confusion with markdown in the listed item-

- [`chat.stopStream`](/reference/methods/chat.stopstream)

The Node Slack SDK provides a [`chatStream()`](/tools/node-slack-sdk/reference/web-api/classes/WebClient#chatstream) helper utility to streamline calling these methods. Here's an excerpt from our [Assistant template app](https://github.com/slack-samples/bolt-js-assistant-template)
The Node Slack SDK provides a [`chatStream()`](/tools/node-slack-sdk/reference/web-api/classes/WebClient#chatstream) helper utility to streamline calling these methods. Here's an excerpt from our [Assistant template app](https://github.com/slack-samples/bolt-js-assistant-template):
Copy link
Member Author

Choose a reason for hiding this comment

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

🗣️ note: This is the big change of the PR!

@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.40%. Comparing base (5f8178f) to head (c9782cb).
⚠️ Report is 15 commits behind head on ai-apps.

Additional details and impacted files
@@           Coverage Diff            @@
##           ai-apps    #2673   +/-   ##
========================================
  Coverage    93.40%   93.40%           
========================================
  Files           37       37           
  Lines         7613     7613           
  Branches       668      668           
========================================
  Hits          7111     7111           
  Misses         497      497           
  Partials         5        5           

☔ View full report in Codecov by Sentry.
📢 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.

Copy link

@srtaalej srtaalej left a comment

Choose a reason for hiding this comment

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

🙌

@zimeg
Copy link
Member Author

zimeg commented Oct 6, 2025

@haleychaas @srtaalej Thanks for the kind reviews! Let's merge this for a publish soon 🚢 💨

@zimeg zimeg merged commit 453ddcc into ai-apps Oct 6, 2025
19 checks passed
@zimeg zimeg deleted the zimeg-docs-ai-colon branch October 6, 2025 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs M-T: Documentation work only semver:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants