- Notifications
You must be signed in to change notification settings - Fork668
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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
jarlah commentedJan 15, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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 commentedJan 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 associatedPR 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!