fallahn / xygine

2D engine / framework built around SFML
218 stars 19 forks source link

Cmake template improvements #101

Closed JonnyPtn closed 6 years ago

JonnyPtn commented 6 years ago

Some things I find myself repeatedly doing whenever I start a new project with xygine, and figured would be useful to add to the template:

fallahn commented 6 years ago

Thanks! The only comment I might make is that MyFirstState is something I'd probably end up renaming when creating a new project, undoing some of the usefulness. That being said I don't have a much better suggestion other than perhaps DefaultState ..? Either way the only reason I've not merged this is the fact that it fails the CI - and I think it's an issue with travis rather than the code itself, so I'm waiting to see if it gets resolved.

JonnyPtn commented 6 years ago

Yeah I always rename it too, it's more useful to save me having to remember which functions I need/want to overload

hat apt-get error on travis is usually just some odd hiccup. I've restarted the build which should solve it, let's see... Travis builds now