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

Fix pypi classifiers #34

Closed
pquentin opened this issue Jun 19, 2015 · 5 comments
Closed

Fix pypi classifiers #34

pquentin opened this issue Jun 19, 2015 · 5 comments

Comments

@pquentin
Copy link
Contributor

Your setup.py file does not define any pypi classifier, which I think prevents pip3 install --pre langid. Here is the list: http://pypi.python.org/pypi?%3Aaction=list_classifiers. I think we should start with:

Programming Language :: Python :: 2
Programming Language :: Python :: 2.7
Programming Language :: Python :: 3

This would fix the Python 3 issue. But since we're on such a good start, we could continue.

You then need to choose a status. Either Beta or Production/Stable, probably Production/Stable.

Development Status :: 4 - Beta
Development Status :: 5 - Production/Stable

Then, an intended audience. I suggest:

Intended Audience :: Developers
Intended Audience :: Science/Research

Then the license. Is your custom license OSI Approved?

And finally a topic:

Topic :: Scientific/Engineering :: Artificial Intelligence

Once you tell me your choices, I can send a pull request. Thanks!

@saffsd
Copy link
Owner

saffsd commented Jun 21, 2015

Thanks for volunteering to help!

I think Production/Stable is fine for this project.

The license is not meant to be custom, it is meant to be BSD 2-Clause. Please let me know if that's not set up right.

With respect to Topic, would

Topic :: Text Processing :: Linguistic

Also apply? I'm not familiar with how these identifiers are used.

@saffsd saffsd mentioned this issue Jun 21, 2015
pquentin added a commit to pquentin/langid.py that referenced this issue Jun 22, 2015
@pquentin
Copy link
Contributor Author

Sure, the topic also seems to apply.

Sorry for the license, I chose 'BSD' as was done in the arguments to setup(), since there's no way to choose between 2-clause and 3-clause.

@pquentin
Copy link
Contributor Author

@saffsd any progress on this?

@saffsd
Copy link
Owner

saffsd commented Aug 12, 2015

Sorry for the delay, it fell off my radar. Going to try and sort #35 now.

@pquentin
Copy link
Contributor Author

Thank you @saffsd!

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

No branches or pull requests

2 participants