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

File bugs against projects missing trove classifier in overrides.json #15

Closed
brettcannon opened this Issue Feb 21, 2014 · 10 comments

Comments

3 participants
@brettcannon
Copy link
Owner

brettcannon commented Feb 21, 2014

@brettcannon brettcannon added the task label Feb 21, 2014

@brettcannon

This comment has been minimized.

Copy link
Owner

brettcannon commented Feb 23, 2014

For mox3 I have asked on Twitter to find out where the source code is: https://twitter.com/brettsky/status/437629859880112128

@brettcannon

This comment has been minimized.

Copy link
Owner

brettcannon commented Feb 23, 2014

@Steap

This comment has been minimized.

Copy link

Steap commented Feb 27, 2014

@brettcannon: about pbr, I don't think OpenStack takes Github pull requests. You should a patch to review.openstack.org using gerrit :)

@brettcannon

This comment has been minimized.

Copy link
Owner

brettcannon commented Feb 27, 2014

@Steap That's a bit more work than I'm willing to do. =)

@Steap

This comment has been minimized.

Copy link

Steap commented Feb 27, 2014

If you do not mind, I can forward the patch using the usual workflow; not sure I can leave your name in the "Author" field (as you haven't signed the pile of paperwork), but I can credit you in the commit log. WDYT ?

@brettcannon

This comment has been minimized.

Copy link
Owner

brettcannon commented Feb 27, 2014

@Steap SGTM

@Steap

This comment has been minimized.

Copy link

Steap commented Feb 28, 2014

python-keystoneclient 0.6.0 works with Python 3. I sent this patch:

https://review.openstack.org/#/c/74543/

Could you add it to overrides.json ?

brettcannon added a commit that referenced this issue Feb 28, 2014

@brettcannon

This comment has been minimized.

Copy link
Owner

brettcannon commented Feb 28, 2014

In case GitHub didn't notify you, @Steap , I made the addition. Thanks for the upstream patch!

@brettcannon

This comment has been minimized.

Copy link
Owner

brettcannon commented Feb 28, 2014

I have done my due diligence, so I consider this bug closed. If I do a lightning talk at PyCon this year on CIU I will consider publicly shaming projects in overrides.json who have not fixed their stuff. =)

@CyrilRoelandteNovance

This comment has been minimized.

Copy link
Contributor

CyrilRoelandteNovance commented Mar 5, 2014

Alex Gaynor's patch for pbr has been merged: https://review.openstack.org/#/c/75868/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment