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

Unify ingame message prefixes. #12923 #12974

Merged
merged 1 commit into from Mar 19, 2017

Conversation

Projects
None yet
4 participants
@rob-v
Contributor

rob-v commented Mar 16, 2017

Used "Battlefield Control" instead of "Server" or ""

@pchote

This comment has been minimized.

Show comment
Hide comment
@pchote

pchote Mar 18, 2017

Member

Please squash the fixup commit into the first one.

Member

pchote commented Mar 18, 2017

Please squash the fixup commit into the first one.

Unify ingame message prefixes. #12923
Unify ingame message prefixes. #12923
@reaperrr

This comment has been minimized.

Show comment
Hide comment
@reaperrr

reaperrr Mar 19, 2017

Contributor

Please squash the fixup commit into the first one.

He did.
@rob-v For future reference, it's better if you give a short "Updated" notice so every participant/subscriber to your PR is notified when you update it.

Contributor

reaperrr commented Mar 19, 2017

Please squash the fixup commit into the first one.

He did.
@rob-v For future reference, it's better if you give a short "Updated" notice so every participant/subscriber to your PR is notified when you update it.

@abcdefg30 abcdefg30 merged commit cc67f93 into OpenRA:bleed Mar 19, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@abcdefg30

This comment has been minimized.

Show comment
Hide comment
@abcdefg30
Member

abcdefg30 commented Mar 19, 2017

@rob-v rob-v deleted the rob-v:IngameMessagePrefixes branch Apr 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment