Skip to content

Utf8supportforcommand #143

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

Merged

Conversation

CrazyRedScorp
Copy link
Contributor

Support for unicode characters in commands. (try number two)

@CrazyRedScorp
Copy link
Contributor Author

Again =((
Sorry.
How to delete this pull?

@CrazyRedScorp CrazyRedScorp deleted the utf8supportforcommand branch April 14, 2016 18:55
@noplanman
Copy link
Member

This PR looked fine! Not sure why you closed it and deleted your branch 😕
For your next one, you can remove the commented lines too! No need for those any more 😉

@CrazyRedScorp
Copy link
Contributor Author

Fine?! I had a red message with the same error. Ok. I try again :)

@CrazyRedScorp CrazyRedScorp restored the utf8supportforcommand branch April 14, 2016 19:40
@CrazyRedScorp CrazyRedScorp reopened this Apr 14, 2016
@CrazyRedScorp
Copy link
Contributor Author

I clicked restore branch and reopen pull request. Check it, please.

@noplanman
Copy link
Member

Right, you can remove the commented lines, just delete them completely.

Then check here to see why the automated tests failed.
We're using the PSR2 standard for this library, which means that the way the code is styled is important.

@CrazyRedScorp
Copy link
Contributor Author

CrazyRedScorp commented Apr 14, 2016

Wow. I understand now. I will correct now. Thank you very much!

All checks have passed!

@MBoretto
Copy link
Collaborator

Thanks!

@MBoretto MBoretto merged commit 1cb65a1 into php-telegram-bot:develop Apr 14, 2016
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.

3 participants