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

Tests and release tweaks #98

Merged
merged 18 commits into from
May 17, 2020
Merged

Tests and release tweaks #98

merged 18 commits into from
May 17, 2020

Conversation

Code0x58
Copy link
Contributor

@Code0x58 Code0x58 commented Aug 4, 2018

This adds some changes to help with with dev/releases:

  • get the package version from git, rather than being hard coded
  • add package classifiers
  • use Travis to run smoke tests (you'll have to set it up, but it is trivial)
  • build package wheel
  • replace shell script so the package will work with pipsi
  • only recommend installing with pipsi or pip

The package may be compatible with 2.7 and 3.3+, versions lower than that will at least have syntax issues - simple colouring works in these versions (covered by smoke tests).

This also changes what happens when you run publish from setup.py, it will now build a wheel instead of a source distribution.

I noticed there have been a couple of tagged releases, but no corresponding releases on pypi - if you'd like a hand with it let me know.

@Code0x58 Code0x58 changed the base branch from master to develop August 4, 2018 18:28
@Code0x58
Copy link
Contributor Author

I just rebased and sorted out the merge conflicts along the way.

@Code0x58
Copy link
Contributor Author

Code0x58 commented Sep 1, 2018

Is there any feedback on this? I'd quite like to see the console_scripts changes get out in a pypi release so people can just do pipsi install colout to get and use this.

@Code0x58
Copy link
Contributor Author

Poking as I'd like to get this cleared away

@nojhan nojhan merged commit 4e35011 into nojhan:develop May 17, 2020
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.

3 participants