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

missions and tutorial conditions #205

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tomsoftware
Copy link
Contributor

for mission.cc

  • possibility to define winning conditions for a mission or tutorial.
  • possibility to define mission instructions
  • possibility to define mission win/lose screen
    1
    2
    3
    this is a replacement for the pull Mission and tutorial #200

@jonls
Copy link
Member

jonls commented Aug 1, 2016

Thanks @tomsoftware, this looks interesting. You have a lot cpplint checks that failed with your changes. You can see those by clicking through to the Travis CI build log (or click here: https://travis-ci.org/freeserf/freeserf/jobs/143978267 or run cpplint on your own machine: https://github.com/google/styleguide).

Are the victory conditions that are implemented for the tutorial maps the same as in the original game?

@tomsoftware
Copy link
Contributor Author

The victory conditions for the tutorial maps are the same as in the original game without the last level (The last one has enemy towers without a castle).

I know of the code-style-checks-errors. Also the clang compiler doesn't like my scoped enum defines :-( (may it is not configured to alow the use of C++11). Also there are some bugs in my code that need to be fixed but I don't find time to continue...

I like your project very much but I'm not sure if I help or only product additional work for you... thanks so far! I also started with a simple AI but I'm not sure if this is worth shown here... but maybe someone could use this as a start?

Thank you!

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.

None yet

2 participants