Skip to content

Conversation

@lukegalbraithrussell
Copy link
Contributor

Summary

Testing

Category

  • slack_sdk.web.WebClient (sync/async) (Web API client)
  • slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
  • slack_sdk.socket_mode (Socket Mode client)
  • slack_sdk.signature (Request Signature Verifier)
  • slack_sdk.oauth (OAuth Flow Utilities)
  • slack_sdk.models (UI component builders)
  • slack_sdk.scim (SCIM API client)
  • slack_sdk.audit_logs (Audit Logs API client)
  • slack_sdk.rtm_v2 (RTM client)
  • /docs (Documents)
  • /tutorial (PythOnBoardingBot tutorial)
  • tests/integration_tests (Automated tests for this library)

Requirements

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes.

@lukegalbraithrussell lukegalbraithrussell marked this pull request as ready for review October 3, 2025 18:56
@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (ai-apps@1a1b2b3). Learn more about missing BASE report.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             ai-apps    #1764   +/-   ##
==========================================
  Coverage           ?   85.15%           
==========================================
  Files              ?      115           
  Lines              ?    13068           
  Branches           ?        0           
==========================================
  Hits               ?    11128           
  Misses             ?     1940           
  Partials           ?        0           

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

Copy link
Contributor

@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.

⭐ LGTM!

Copy link
Member

@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.

@lukegalbraithrussell LGTM! It's neat to find the expanded headings on how a message can be sent! 📚 ✨

I left a few comments that we might want as suggestion in reference, but this is reading so well already.

@zimeg zimeg added docs M-T: Documentation work only semver:minor Version: 3x labels Oct 3, 2025
Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

✅ Looking good and just a +1 to all of @zimeg's comments!

@lukegalbraithrussell lukegalbraithrussell merged commit 28b191e into ai-apps Oct 3, 2025
12 checks passed
@lukegalbraithrussell lukegalbraithrussell deleted the docs-ai-apps branch October 3, 2025 22:07
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:minor Version: 3x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants