-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Get 1.19 branch going #47
Conversation
S14: Malakar no longer comments on gargoyles if player completed S6B
Avoids wmllint error
also ignore map-diff stuff in `_server.ign`
1.18 upstream update
Looks like one of us might need to start our 1.19 branch over... |
scenarios_b/08b_Apocalypse.cfg
Outdated
# team_name=evil | ||
# user_team_name= _ "Wesnoth Expeditionary Force" | ||
# hidden=yes | ||
#[/side] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally I prefer ifdef-ing unused code out, instead of commenting it out... or, just removing it would be fine, too...
units.cfg
Outdated
# ignore_global_traits=yes | ||
# {TRAIT_MECHANICAL} | ||
# {HUMAN_NAMES} | ||
#[/race] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, I'd prefer ifdef-ing this out, or removing it entirely, instead of commenting it out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few comments/questions, but other than that, OK (looks like I can't approve myself since I opened this myself, even though it's technically your PR, lol)
Just getting this started; it can come out of draft mode once it's ready...
Closes #11 upon merge.