-
-
Notifications
You must be signed in to change notification settings - Fork 995
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
Avoid use of 'cool' and 'guys' (closes #6107) #6108
Conversation
Have left 'guys' in Camerin's lines as it's justifiable in his informal crazy talk - Northern Rebirth is supposed to be rather informal in parts and light-hearted in tone. [ci skip]
I'm not sure if it's your fault or GitHub's, but the issue link in your commit message is broken… it seems to link to your fork instead of the main Wesnoth repo. |
Yea, reading them over, the NR ones especially feel too formal to me. Personally I wouldn't make this change, but depends on what your goal is. Edit: just for NR, the other ones are fine. |
I think it's GitHub. I haven't done anything differently but I've noticed it's been showing inconsistently recently to me. Edit: It reads as linking to the main wesnoth/wesnoth issue to me. |
@nemaara Can you highlight which ones you object to? Is it all of the Northern Rebirth ones? |
data/campaigns/Northern_Rebirth/scenarios/05a_01_The_Pursuit.cfg
Outdated
Show resolved
Hide resolved
data/campaigns/Northern_Rebirth/scenarios/05a_01_The_Pursuit.cfg
Outdated
Show resolved
Hide resolved
data/campaigns/Northern_Rebirth/scenarios/05a_01_The_Pursuit.cfg
Outdated
Show resolved
Hide resolved
Avoid use of 'cool' and 'guys' (closes wesnoth#6107) Have left 'guys' in Camerin's lines as it's justifiable in his informal crazy talk - Northern Rebirth is supposed to be rather informal in parts and light-hearted in tone. [ci skip]
As discussed in #6107, have left 'guys' in Camerin's lines alone.