Skip to content

Slack client v2 development#394

Merged
RodneyU215 merged 75 commits into
masterfrom
v2
Apr 29, 2019
Merged

Slack client v2 development#394
RodneyU215 merged 75 commits into
masterfrom
v2

Conversation

@RodneyU215

@RodneyU215 RodneyU215 commented Mar 29, 2019

Copy link
Copy Markdown
Contributor

Summary

This PR is a WIP. It implements most of the changes specified in #384. However, the most notable deviation is that we've decided that v2 will target Python 3.6+ only! Python 2.7+ will continue to be supported in v1 until Dec 31st, 2019. See additional details in #384!

Still in development:

  • Type Hints for parameters and return values.
  • Unit & Integration Testing
  • ensure proper logging levels throughout project.
  • Readme Updates
  • Documentation Overhaul (https://slack.dev/python-slackclient/)
  • 1.x Migration Guide
  • FAQ / Troubleshooting Guide. (i.e. SSL Issues & Proxy Support)
  • Updated Example Apps and Snippets

Requirements

@codecov

codecov Bot commented Apr 8, 2019

Copy link
Copy Markdown

Codecov Report

Merging #394 into master will decrease coverage by 19.58%.
The diff coverage is 57.16%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #394       +/-   ##
===========================================
- Coverage   76.55%   56.96%   -19.59%     
===========================================
  Files          10        6        -4     
  Lines         418      732      +314     
  Branches       75       42       -33     
===========================================
+ Hits          320      417       +97     
- Misses         87      306      +219     
+ Partials       11        9        -2
Impacted Files Coverage Δ
slack/version.py 100% <100%> (ø)
slack/errors.py 100% <100%> (ø)
slack/web/client.py 33.56% <33.56%> (ø)
slack/web/base_client.py 84.41% <84.41%> (ø)
slack/rtm/client.py 90% <90%> (ø)
slack/web/slack_response.py 97.67% <97.67%> (ø)
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66f1fb5...a0a2232. Read the comment docs.

@RodneyU215 RodneyU215 marked this pull request as ready for review April 8, 2019 05:28
@slackapi slackapi deleted a comment from codecov Bot Apr 27, 2019
@codecov

codecov Bot commented Apr 29, 2019

Copy link
Copy Markdown

Codecov Report

Merging #394 into master will decrease coverage by 19.58%.
The diff coverage is 57.16%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #394       +/-   ##
===========================================
- Coverage   76.55%   56.96%   -19.59%     
===========================================
  Files          10        6        -4     
  Lines         418      732      +314     
  Branches       75       42       -33     
===========================================
+ Hits          320      417       +97     
- Misses         87      306      +219     
+ Partials       11        9        -2
Impacted Files Coverage Δ
slack/version.py 100% <100%> (ø)
slack/errors.py 100% <100%> (ø)
slack/web/client.py 33.56% <33.56%> (ø)
slack/web/base_client.py 84.41% <84.41%> (ø)
slack/rtm/client.py 90% <90%> (ø)
slack/web/slack_response.py 97.67% <97.67%> (ø)
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66f1fb5...1eee78e. Read the comment docs.

@RodneyU215 RodneyU215 merged commit 72f4f1e into master Apr 29, 2019
c-goosen pushed a commit to c-goosen/python-slackclient that referenced this pull request Jun 18, 2019
This PR implements the changes specified in RFC slackapi#384. However, the most notable deviation is that we've decided that v2 will target Python 3.6+ only! Python 2.7+ will continue to be supported in v1 until Dec 31st, 2019. See additional details in slackapi#384!
@RodneyU215 RodneyU215 deleted the v2 branch October 29, 2019 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants