Skip to content

Conversation

@staticdev
Copy link

Summary

Quick fix for issue #433. It transforms install_requires of aiodns into extras_require.

Reference documentation: https://setuptools.readthedocs.io/en/latest/setuptools.html

Requirements (place an x in each [ ])

@CLAassistant
Copy link

CLAassistant commented Jun 11, 2019

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Jun 12, 2019

Codecov Report

Merging #440 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #440   +/-   ##
=======================================
  Coverage   56.94%   56.94%           
=======================================
  Files           6        6           
  Lines         734      734           
  Branches       42       42           
=======================================
  Hits          418      418           
  Misses        307      307           
  Partials        9        9

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 897807f...7e5fc02. Read the comment docs.

@RodneyU215 RodneyU215 added Priority: Medium enhancement M-T: A feature request for new functionality Version: 2x labels Jun 13, 2019
@RodneyU215 RodneyU215 self-assigned this Jun 13, 2019
Copy link
Contributor

@RodneyU215 RodneyU215 left a comment

Choose a reason for hiding this comment

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

This is exactly what we need! Thank you!

@RodneyU215 RodneyU215 merged commit cedbadc into slackapi:master Jun 13, 2019
c-goosen pushed a commit to c-goosen/python-slackclient that referenced this pull request Jun 18, 2019
@RodneyU215 RodneyU215 mentioned this pull request Jul 1, 2019
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality Version: 2x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants