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

Stderr output cuts itself #148

Closed
Iinksafe opened this issue Apr 25, 2022 · 4 comments · Fixed by #150
Closed

Stderr output cuts itself #148

Iinksafe opened this issue Apr 25, 2022 · 4 comments · Fixed by #150
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@Iinksafe
Copy link

Iinksafe commented Apr 25, 2022

Hi there! So I was compiling a code I made today and was going to test how it works on Compiler!

Basically, the standard error output (stderr) cuts itself in the embed's description.

I have attached an image for further understanding.
IMG_20220425_015428

@es183923
Copy link

It is defined here: https://github.com/Headline/discord-compiler-bot/blob/master/src/utls/constants.rs#L15-L16
Discord limits embed values to 1024 characters, which is way more than the 500/250 char limit defined here. https://discord.com/developers/docs/resources/channel#embed-object-embed-limits

@Headline Headline added enhancement New feature or request good first issue Good for newcomers labels Apr 25, 2022
@Headline
Copy link
Owner

Headline commented Apr 25, 2022

Languages which have verbose outputs over our limit will always get truncated. The limit could get increased as mentioned. If someone beats me to it, we shouldn't be up against the limit too much, give us a hundred characters to play with.

As for the output limit - it should remain at 250 to prevent channel spam

@Iinksafe
Copy link
Author

Alright, thank you for the quick response!

@Headline Headline reopened this Apr 27, 2022
@Headline
Copy link
Owner

I'd like to leave this open - we'll increase the error output limit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants