Skip to content

chore: remove experiment around say_stream - #1471

Merged
zimeg merged 2 commits into
mainfrom
remove-say-stream-experiement
Apr 6, 2026
Merged

chore: remove experiment around say_stream#1471
zimeg merged 2 commits into
mainfrom
remove-say-stream-experiement

Conversation

@WilliamBergamin

Copy link
Copy Markdown
Contributor

Summary

These changes aim to remove the experiment warning around say stream

Testing

CI should be sufficient

Category

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter
  • Document pages under /docs
  • Others

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

  • 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 ./scripts/install_all_and_run_tests.sh after making the changes.

@WilliamBergaminWilliamBergamin added this to the 1.28.0 milestone Mar 26, 2026
@WilliamBergaminWilliamBergamin self-assigned this Mar 26, 2026
@WilliamBergamin
WilliamBergamin requested a review from a team as a code ownerMarch 26, 2026 17:41
@WilliamBergaminWilliamBergamin added area:async area:sync python Pull requests that update Python code semver:patch experiment Experimental feature documented with ExperimentalWarning and pydoc Experiment section labels Mar 26, 2026
@codecov

codecovBot commented Mar 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.31%. Comparing base (dbe1590) to head (96afa5d).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@ Coverage Diff @@## main #1471 +/- ##
==========================================
- Coverage 91.32% 91.31% -0.01% 
==========================================
Files 229 228 -1 Lines 7270 7262 -8 ==========================================
- Hits 6639 6631 -8 
Misses 631 631 

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

@zimegzimeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@WilliamBergamin LGTM! When the time is right let's get this merged 🚀 ✨

I look forward to using this in stable code.

@zimeg
zimeg merged commit 064ef2e into mainApr 6, 2026
16 checks passed
@zimeg
zimeg deleted the remove-say-stream-experiement branch April 6, 2026 21:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:asyncarea:syncexperimentExperimental feature documented with ExperimentalWarning and pydoc Experiment sectionpythonPull requests that update Python codesemver:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@WilliamBergamin@zimeg