-
Notifications
You must be signed in to change notification settings - Fork 673
Add elixir as a language option #731
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
Conversation
@Weilet
Weilet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
can we get this merged? this is so simple, if its the only thing needed ..... i wouldn't even call this necroing .. this is just .. unnecessary :D
jarlah
commented
Jan 15, 2025
all in all .. this stops me from using this extension . no elixir. No leetcode. I dont want to fiddle around with leetcode-cli. I even see that also lacks elixir ...
Uh oh!
There was an error while loading. Please reload this page.
I saw that elixir was now supported in the leetcode web UI, and thought it would be nice to be able to do that within vscode. This PR basically just adds elixir anywhere else that has boilerplate for other languages. I wasn't sure the best way to test this, so I deleted
~/.vscode/extensions/leetcode.vscode-leetcode-0.18.0/outand~/.vscode/extensions/leetcode.vscode-leetcode-0.18.0/node_modules/vsc-leetcode-cliand replaced the contents with my forked repos. After that I was able to set my default language and generate/test/submit problems all in elixir.This work will depend on an update the the cli repo dependency. See associated PR here. So we'll need to update that dependency once the other PR is merged/released. If there's anything missing from this PR or if there's anything else I can provide to help with this!