Skip to content

fix: configure dependabot to group react deps#1664

Merged
WilliamBergamin merged 1 commit intomainfrom
group-react-deps-for-dependabot-docs
Mar 3, 2025
Merged

fix: configure dependabot to group react deps#1664
WilliamBergamin merged 1 commit intomainfrom
group-react-deps-for-dependabot-docs

Conversation

@WilliamBergamin
Copy link
Copy Markdown
Contributor

Summary

#1645 and #1644 are failing tests because dependabot is opening separate PRs for react and react-dom dependencies, I believe these dependencies must stay in sync in order for the tests to pass.

Once this PR is merged dependabot should open one single PR to replace #1645 and #1644

Testing

n/a

Category

  • slack_sdk.web.WebClient (sync/async) (Web API client)
  • slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
  • slack_sdk.socket_mode (Socket Mode client)
  • slack_sdk.signature (Request Signature Verifier)
  • slack_sdk.oauth (OAuth Flow Utilities)
  • slack_sdk.models (UI component builders)
  • slack_sdk.scim (SCIM API client)
  • slack_sdk.audit_logs (Audit Logs API client)
  • slack_sdk.rtm_v2 (RTM client)
  • /docs (Documents)
  • /tutorial (PythOnBoardingBot tutorial)
  • tests/integration_tests (Automated tests for this library)

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 python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes.

@WilliamBergamin WilliamBergamin added docs M-T: Documentation work only dependency-side-issue github_actions Pull requests that update GitHub Actions code labels Mar 3, 2025
@WilliamBergamin WilliamBergamin requested a review from mwbrooks March 3, 2025 22:21
@WilliamBergamin WilliamBergamin self-assigned this Mar 3, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.36%. Comparing base (161c921) to head (4998ffe).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1664      +/-   ##
==========================================
+ Coverage   85.33%   85.36%   +0.03%     
==========================================
  Files         113      113              
  Lines       12802    12802              
==========================================
+ Hits        10924    10928       +4     
+ Misses       1878     1874       -4     

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

Copy link
Copy Markdown
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

👌🏻

@WilliamBergamin WilliamBergamin merged commit 931dcd6 into main Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependency-side-issue docs M-T: Documentation work only github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants