Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feature: Added an option to disable endpoint translation#690

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

Merged
jdneo merged 8 commits intoLeetCode-OpenSource:masterfromdzz007:develop
May 31, 2021

Conversation

dzz007
Copy link
Contributor

After my pull request gets merged on leetcode-cli (which provides the -T option to disable endpoint translation), this pull requests will give user an option to toggle in settings whether or not they want to use endpoint's translation for the problems.

@dzz007
Copy link
ContributorAuthor

@jdneo Thank you for bumping leetcode-cli to 2.8.0! Do you mind publish that change to npmjs as well? So I can specify the new version in package.json. Or do you prefer to specify the commit hash in package.json?

@jdneo
Copy link
Member

@dzz007 you can first use the latest bit of the cli and link it to the vs code extension locally. Let me know if everything works fine. After that I'll release it to the registry.

@dzz007
Copy link
ContributorAuthor

@jdneo thank you for responding! I just tested with locally installing with master branch of leetcode-cli in vscode-leetcode. Everything works fine, I checked both the leetcode.com and leetcode-cn.com endpoints.
The only problem I found is that changing Endpoint in vscode-leetcode's setting page actually doesn't work. But that is a separate bug which is not related to this leetcode-cli upgrade at all, and I verified that because it exists in the current version as well.

@jdneo
Copy link
Member

Just published 2.8.0.

@dzz007
Copy link
ContributorAuthor

@jdneo Thanks, just finished changing package.json & package-lock.json. I tested again and it works well.

@jdneo
Copy link
Member

The content of package-lock.json looks strange, why the entries havenode_modules prefix?

@dzz007
Copy link
ContributorAuthor

I believe it's because I am currently using NPM v7 which introduced some changes to the lock file as specified in "lockfileVersion": 2. I can downgrade to NPM v6 and regenerate the lock file if you want.

@dzz007
Copy link
ContributorAuthor

Just a sidenote, IIRC v2 version of the lock-file is backward compatible with older version of NPM. So it should be able to work fine with NPMv6 for example.

@jdneo
Copy link
Member

ok, thanks for the information

@dzz007
Copy link
ContributorAuthor

@jdneo no problem! Do you want me to keep the lock file as-is or do you want or to use the old version?

@jdneo
Copy link
Member

If it's backward compatible, then it's fine I think.

@dzz007
Copy link
ContributorAuthor

Thank you for your review! I just added the application scope.

yihong0618
yihong0618 previously approved these changesApr 14, 2021
@dzz007dzz007 requested a review fromjdneoApril 14, 2021 17:56
@jdneo
Copy link
Member

Coule you add a tsdoc for theshowSolution function notifying the param needTranslation will not take effect?

@dzz007
Copy link
ContributorAuthor

@jdneo That's a good point! I will add it in couple hours. Do you want me to put it in leetCodeExecutor?

@dzz007
Copy link
ContributorAuthor

Thanks!

@jdneojdneo merged commit0fcf9f1 intoLeetCode-OpenSource:masterMay 31, 2021
@jdneojdneo added this to the0.17.1 milestoneMay 31, 2021
@jdneo
Copy link
Member

Apologize for lazy merge. I just get the permission to merge the PRs. 😃

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@jdneojdneojdneo approved these changes

@yihong0618yihong0618yihong0618 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.18.0
Development

Successfully merging this pull request may close these issues.

3 participants
@dzz007@jdneo@yihong0618

[8]ページ先頭

©2009-2025 Movatter.jp