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

Correct typos ("enginer" to "engineer") #5355

Merged
merged 4 commits into from
Jul 11, 2017
Merged

Conversation

Wakbub
Copy link
Contributor

@Wakbub Wakbub commented Jul 11, 2017

When merged this pull request will:

  • Correct typos ("enginer" to "engineer").

Wakbub added 4 commits July 11, 2017 04:03
Correction of typos.
Correction of typos.
Correction of typos.
In accordance with your process of merging pull requests.
@PabstMirror PabstMirror added this to the 3.11.0 milestone Jul 11, 2017
@PabstMirror PabstMirror added the kind/cleanup Release Notes: **CHANGED:** label Jul 11, 2017
@PabstMirror PabstMirror changed the title Typos Correct typos ("enginer" to "engineer") Jul 11, 2017
@bux
Copy link
Member

bux commented Jul 11, 2017

literally unplayable 😂

@Wakbub
Copy link
Contributor Author

Wakbub commented Jul 11, 2017

Forsooth, literally unplayable for non-engineers in missions which depend on repairing vehicles and use properly spelled Key IDs from the string tables of ACE in its mission parameters, without incredible guesswork what the missing parameter names really represent.

@jonpas
Copy link
Member

jonpas commented Jul 11, 2017

And you have a white GitHub avatar, this is the second time I opened this PR from notifications and wondered what the heck is wrong.

@PabstMirror PabstMirror merged commit 2c97865 into acemod:master Jul 11, 2017
@commy2
Copy link
Contributor

commy2 commented Jul 11, 2017

Stole my Steam avatar.

@Wakbub Wakbub deleted the typos branch July 11, 2017 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Release Notes: **CHANGED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants