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

Add parse mode to options #17

Closed
wants to merge 3 commits into from

Conversation

golubvladimir
Copy link

Add parse mode to options for formatting messages.

@ivanmarban
Copy link
Owner

What is the point of this PR ? I do not see the necesity of formatting messages in Markdown or HTML, as you are adding extra characters to the final message to be sent. This could be a problem with the length of the message.

@golubvladimir
Copy link
Author

@ivanmarban I want to style the message

*name error*: ${ name }

@golubvladimir
Copy link
Author

golubvladimir commented Dec 11, 2019

@ivanmarban
Will be accepted PR ?

@ivanmarban
Copy link
Owner

Closing this PR as is not going to be merged. Sorry.

@ivanmarban ivanmarban closed this Dec 12, 2019
@golubvladimir
Copy link
Author

@ivanmarban why ?

@ivanmarban
Copy link
Owner

As I said, I don't see the necessity of formatting log messages in Markdown or HTML

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.

2 participants