Skip to content

docs(deps): add the managing-dependencies skill and apply its layout - #137

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

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

Conversation

@WilliamBergamin

@WilliamBergamin WilliamBergamin commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds the managing-dependencies skill that bolt-python and python-slack-sdk already have, and puts this repo's requirements files in its layout.

What's here:

  • New skill: .claude/skills/managing-dependencies/SKILL.md. It handles Dependabot bumps that fail on only some Pythons, like build(deps): update gevent requirement from <27,>=26.7.0 to >=26.9.0,<27 #136 (gevent 26.9.0 needs Python 3.10+).
  • Requirements layout: a # pip install -r header and one section per package in requirements.txt and requirements/*.txt. Also drops a stray ; after flake8 and a trailing space after Flask.
  • .claude/ is now tracked. Removed it from .gitignore and added .claude/.gitignore for local-only files, as the other two repos do.

Testing

n/a

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

🤖 Generated with Claude Code

Adds the skill shared with bolt-python and python-slack-sdk, and puts
the requirements files in its layout (a pip install header, one section
per package). No version constraint changes. .claude/ is now tracked,
with local-only files ignored by .claude/.gitignore as in the other two
repos.

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 94.68%. Comparing base (0ad0b37) to head (5ab8223).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #137   +/-   ##
=======================================
  Coverage   94.68%   94.68%           
=======================================
  Files          11       11           
  Lines         207      207           
=======================================
  Hits          196      196           
  Misses         11       11           

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

@WilliamBergamin
WilliamBergamin marked this pull request as ready for review September 25, 2026 19:23
@WilliamBergamin
WilliamBergamin requested a review from a team as a code owner September 25, 2026 19:23
@WilliamBergamin
WilliamBergamin merged commit 9214784 into main Sep 25, 2026
19 checks passed
@WilliamBergamin
WilliamBergamin deleted the add-managing-dependencies-skill branch September 25, 2026 19:55
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