Skip to content

chore: rename requirements files to improve Dependabot dependency scoping - #1510

Merged
WilliamBergamin merged 1 commit into
mainfrom
imporve-dependency-scope
May 27, 2026
Merged

chore: rename requirements files to improve Dependabot dependency scoping#1510
WilliamBergamin merged 1 commit into
mainfrom
imporve-dependency-scope

Conversation

@WilliamBergamin

@WilliamBergaminWilliamBergamin commented May 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Dependabot is currently including all our development and testing dependencies as production dependencies, this leads to irrelevant Dependabot security alerts

Renaming these files to include dev or test should allow Dependabot to properly scope these dependencies as development and testing

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 self-assigned this May 27, 2026
@WilliamBergamin
WilliamBergamin requested a review from a team as a code ownerMay 27, 2026 18:49
@WilliamBergaminWilliamBergamin added the dependencies Pull requests that update a dependency file label May 27, 2026

@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 and @dependabot Kind thanks for these continued improvements 🎁

@codecov

codecovBot commented May 27, 2026

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@ Coverage Diff @@## main #1510 +/- ##
=======================================
Coverage 91.30% 91.30% =======================================
Files 228 228 Lines 7271 7271 =======================================
Hits 6639 6639 Misses 632 632 

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

@WilliamBergamin
WilliamBergamin merged commit 92d8516 into mainMay 27, 2026
16 checks passed
@WilliamBergamin
WilliamBergamin deleted the imporve-dependency-scope branch May 27, 2026 18:54
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependenciesPull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@WilliamBergamin@zimeg