search

jostbr / pymaze

A maze generator, solver and visualizer for Python
MIT License
272 stars 63 forks source link

Merge #20

Closed ThomasThelen closed 6 years ago

ThomasThelen commented 6 years ago

Sorry I've been super busy with work and holidays-this should do it!

12 Fixed

11 Fixed

3 Fixed

4 Partially Fixed (needs you)

1 Fixed

Don't bother looking at the individual commits, it took ~ 3 hours of trial and error to get Travis CI working :/

I added some unit tests for the maze class and re-factored a little of maze_viz. I also went ahead and created a couple of different example files and put them in an examples directory. There is an example for each maze solution type.

To Complete Issue 4

You need to go to https://travis-ci.org and sign in with your github account and add MazeGenerator to the repo list. I should have set everything else up such that it works out of the box (I got it on build 23 :/). Right now we have three unit test failures, but I'll put that into a different issue. In case you didn't know, Travis will run our unit tests every time we commit to master.

We also test this on Python 3.6, Python 3.5, and Python 3.4. To test any more versions that you are interested in, add them in the travis.yaml file. It's great that this all gets automated, and lets us know when anything breaks.

To get Code Climate up and running, head over to https://codeclimate.com and sign in with github. Add the MazeGenerator repo.

In the badges in the readme, I put your name in so that you shouldn't need to copy the markdown over from the sites.

ThomasThelen commented 6 years ago

I also have another commit to push up once this one gets accepted so check back ~1 day after!

jostbr commented 6 years ago

Thanks for the commit. Sorry for being late on merging. Great work. I might have some questions I'll post later.

jostbr commented 6 years ago

If I change the repo name and then enter the new name on travis-ci.org; is that OK?