Skip to content

docs(deps): simplify the managing-dependencies skill - #1578

Merged
WilliamBergamin merged 1 commit into
mainfrom
simplify-managing-dependencies-skill
Sep 25, 2026
Merged

WilliamBergamin merged 1 commit into
mainfrom
simplify-managing-dependencies-skill

Conversation

@WilliamBergamin

@WilliamBergamin WilliamBergamin commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Simplifies the managing-dependencies skill. It quoted live requirement lines and a per-file CI table, and later Dependabot fixes made several of those examples wrong within weeks.

What changed:

  • No repo facts in the skill. It says where to read them instead (pyproject.toml, a grep over workflows and scripts), so it can't go stale.
  • Examples use a fictional pkg.
  • Adds what was missing: 3+ way splits, splitting only the Pythons the bumped line covers, and Dependabot rewriting only the last line of a split section.
  • New one-Python bands use == "3.9". python_version is major.minor, so that covers every 3.9 patch. Existing lines are left alone.
  • Shorter: 129 to 81 lines.

Testing

n/a

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: Claude Code skill (.claude/skills/)

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. (Markdown only, so I'm relying on CI.)

🤖 Generated with Claude Code

The skill quoted live requirement lines and a per-file CI table, which
went stale within weeks. It now points at where each fact lives, uses a
fictional package in its examples, and covers 3+ way splits and
Dependabot rewriting only the last line of a split section. The same
file ships in bolt-python, python-slack-sdk and python-slack-hooks.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@codecov

codecov Bot commented Sep 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.54%. Comparing base (c793795) to head (c1eb4ff).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1578   +/-   ##
=======================================
  Coverage   91.54%   91.54%           
=======================================
  Files         228      228           
  Lines        7285     7285           
=======================================
  Hits         6669     6669           
  Misses        616      616           

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

@WilliamBergamin WilliamBergamin self-assigned this Sep 25, 2026
@WilliamBergamin
WilliamBergamin marked this pull request as ready for review September 25, 2026 19:21
@WilliamBergamin
WilliamBergamin requested a review from a team as a code owner September 25, 2026 19:21
@WilliamBergamin
WilliamBergamin merged commit eddc476 into main Sep 25, 2026
16 checks passed
@WilliamBergamin
WilliamBergamin deleted the simplify-managing-dependencies-skill branch September 25, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant