Skip to content

Add test+build infrastructure (Tox and Travis CI) #100

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

Merged
merged 1 commit into from
Sep 16, 2018

Conversation

bittner
Copy link
Member

@bittner bittner commented Sep 16, 2018

First steps towards automatic builds. Adds:

  • a Tox configuration for running linting and tests (locally), and
  • a Travis CI configuration.

See https://travis-ci.org/bittner/PythonTurtle while I'm testing the results of this implementation.

@bittner bittner force-pushed the feature/ci-configuration branch from e3443ed to b6ea687 Compare September 16, 2018 07:27
@bittner bittner force-pushed the feature/ci-configuration branch from b6ea687 to 71a1693 Compare September 16, 2018 07:30
@cool-RR cool-RR merged commit f503b5d into PythonTurtle:master Sep 16, 2018
@cool-RR
Copy link
Member

cool-RR commented Sep 16, 2018

Great job!

@bittner
Copy link
Member Author

bittner commented Sep 16, 2018

You now need to activate this repository on Travis CI:

  1. Log in to your Travis CI account
  2. Go to https://travis-ci.org/cool-RR/PythonTurtle
  3. Push the "Activate this repository" button

@cool-RR
Copy link
Member

cool-RR commented Sep 16, 2018 via email

@bittner bittner deleted the feature/ci-configuration branch September 16, 2018 07:46
@bittner
Copy link
Member Author

bittner commented Sep 16, 2018

Yeah, that looks good! https://travis-ci.org/cool-RR/PythonTurtle

Next Steps

  1. I'll add a cute build status badge on top of the README for this.
  2. I'll make sure we get a green tick for the build status for new commits and PRs.

Afterwards, I'll try to:

  1. Make the code base comply with PEP8 using flake8 linting.
  2. Fix issues Pylint complains about.

Finally, the less trivial tasks:

  1. Figure out whether I can port the codebase to wxgtk4.0, still staying compatible with wxgtk3.0. (This may be impractical, we'll see.) -- Migrate PythonTurtle to wxgtk4.0 #99
  2. Make the code base work across all Python versions we're testing against (including Python 3 and PyPy).
  3. Add a few tests that actually make sense and help with refactoring.

In the end, we can look into automating the building of the installer packages.

@cool-RR
Copy link
Member

cool-RR commented Sep 16, 2018 via email

@bittner
Copy link
Member Author

bittner commented Sep 16, 2018

Thanks for merging so quickly, by the way! 👍 😃

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.

2 participants