Skip to content

Urban Dictionary: better embed#1127

Merged
Vendicated merged 7 commits into
Vendicated:devfrom
tooslzy:main
May 17, 2023
Merged

Urban Dictionary: better embed#1127
Vendicated merged 7 commits into
Vendicated:devfrom
tooslzy:main

Conversation

@tooslzy
Copy link
Copy Markdown
Contributor

@tooslzy tooslzy commented May 14, 2023

image

@Vendicated
Copy link
Copy Markdown
Owner

you changed the entire formatting. Please fix that

@Vendicated
Copy link
Copy Markdown
Owner

looks to me like you somehow changed the file from LF to CRLF. Not sure how considering we have an .editorconfig and .gitattributes that should fix it, but yeah

@Vendicated
Copy link
Copy Markdown
Owner

oh, it's not CRLF, you just changed it to tabs. pnpm lint:fix should fix it

@tooslzy
Copy link
Copy Markdown
Contributor Author

tooslzy commented May 14, 2023

oh, it's not CRLF, you just changed it to tabs. pnpm lint:fix should fix it

I edited the file on github.dev. I think that's why it messed up.

Comment thread src/plugins/urbanDictionary.ts Outdated
Comment thread src/plugins/urbanDictionary.ts Outdated
Comment thread src/plugins/urbanDictionary.ts Outdated
tooslzy and others added 3 commits May 14, 2023 17:22
Co-authored-by: V <vendicated@riseup.net>
It still works with double quotes, but I'll switch it to single quotes.

Co-authored-by: V <vendicated@riseup.net>
@tooslzy tooslzy changed the title Urban Dictionary: better embed & use highest voted ratio definition Urban Dictionary: better embed May 16, 2023
Comment thread src/plugins/urbanDictionary.ts Outdated
@Vendicated Vendicated changed the base branch from main to dev May 17, 2023 02:40
@Vendicated Vendicated merged commit c2a1c4c into Vendicated:dev May 17, 2023
EtorixDev pushed a commit to EtorixDev/Vencord that referenced this pull request May 29, 2026
* Fix VoiceButtons DM opening

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
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