Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add long description, classifiers and update info in setup.py #465

Merged
merged 2 commits into from
Feb 21, 2018

Conversation

jacobtomlinson
Copy link
Member

Description

I was checking out warehouse and spotted that on the new pypi opsdroid page there is no description and other info. This PR updates the setup.py to add a long description, classifiers (chosen from here) and update other general info.

Fixes NA

Status

READY

Type of change

  • Documentation (fix or adds documentation)

How Has This Been Tested?

Running python3 setup.py install succeeds.

Checklist:

  • I have performed a self-review of my own code
  • New and existing unit tests pass locally with my changes

@codecov
Copy link

codecov bot commented Feb 21, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #465   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          18     18           
  Lines        1095   1095           
=====================================
  Hits         1095   1095

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 88aa0bd...283e2cc. Read the comment docs.

@FabioRosado
Copy link
Member

I still use the old website so I haven't noticed this issue, well spotted!
I also like the new keywords to identify opsdroid 👍

@FabioRosado FabioRosado merged commit b45cb64 into opsdroid:master Feb 21, 2018
@jacobtomlinson jacobtomlinson deleted the update-setup-pypi branch February 21, 2018 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants