Skip to content

Conversation

@misscoded
Copy link
Contributor

Summary

Remove deprecated Steps from Apps feature, including related types, tests, and documentation.

Requirements (place an x in each [ ])

@misscoded misscoded added this to the 5.0.0 milestone Oct 15, 2025
@misscoded misscoded self-assigned this Oct 15, 2025
@misscoded misscoded added semver:major tests M-T: Testing work only labels Oct 15, 2025
@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.09%. Comparing base (9204d43) to head (518cbfe).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2688      +/-   ##
==========================================
- Coverage   93.40%   93.09%   -0.32%     
==========================================
  Files          37       35       -2     
  Lines        7613     7080     -533     
  Branches      668      634      -34     
==========================================
- Hits         7111     6591     -520     
+ Misses        497      484      -13     
  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.

@misscoded
Copy link
Contributor Author

@mwbrooks @haleychaas There are legacy docs that exist for this feature at docs/english/legacy/steps-from-apps.md and docs/japanese/legacy/steps-from-apps.md that I'm unsure whether to remove or not.

@mwbrooks
Copy link
Member

@mwbrooks @haleychaas There are legacy docs that exist for this feature at docs/english/legacy/steps-from-apps.md and docs/japanese/legacy/steps-from-apps.md that I'm unsure whether to remove or not.

Definitely @haleychaas call on how they want to handle the deprecation of docs 🤔

@haleychaas
Copy link
Contributor

@misscoded Is there a way someone could still use it with an old version of Bolt? If so, I'd say we keep the docs and label them as legacy, but if not....I'm tempted to delete.

@misscoded
Copy link
Contributor Author

@haleychaas Technically, no. Even if they were using it in an old version of Bolt, it wouldn't work, as the feature has been shut off. They'd get a deprecation warning and/or errors, and not be able to even see the feature, AFAIK.

Usually I'd say we should keep it for posterity, but it gets dicey because SFA and the new version of steps are so similar – meaning it's possible that someone could happen upon this old stuff and think it's referring to the new stuff. A problem we shouldn't have, but one that we do, unfortunately. 🫠

So, I can see both arguments. If we want to remove it, just let me know - I leave that decision ultimately to the docs folks!

@haleychaas
Copy link
Contributor

@misscoded I say we delete it then! 🪓

@misscoded misscoded force-pushed the remove-steps-from-apps branch from 518cbfe to 6afcc4e Compare November 13, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:major tests M-T: Testing work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants