Skip to content

Update server_software_information.conf #55

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

Closed
wants to merge 1 commit into from

Conversation

alexander-akait
Copy link

No description provided.

@arthurvr
Copy link
Member

Would fix #54.

Thanks for the PR, @sheo13666! We should probably add some comments on why we add this. See the other inline comments as examples. Also, from the original issue:

we should also add a comment explaining the better alternatives.

@arthurvr
Copy link
Member

friendly ping for @sheo13666

@AD7six AD7six added enhancement New feature or request awaiting feedback Further information is requested labels Feb 23, 2015
@alexander-akait
Copy link
Author

sorry my English is bad for detail comments 😞

@Philosoft
Copy link

@sheo13666 напиши по-русски тогда, и я переведу в меру сил)

@alrra
Copy link
Member

alrra commented Feb 27, 2015

sorry my English is bad for detail comments 😞

@sheo13666 I took care of this addition, as it required more then a few comments. Thanks again for bringing this up!

@alrra alrra removed awaiting feedback Further information is requested enhancement New feature or request labels Feb 27, 2015
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.

5 participants