Skip to content

feat: Add javascript Solution2 for 1 #89

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

Merged

Conversation

yaodehaung
Copy link
Contributor

No description provided.

@fishercoder1534
Copy link
Owner

fishercoder1534 commented Jul 14, 2020

@yaodehaung
Copy link
Contributor Author

I'm fixing XD

@fishercoder1534
Copy link
Owner

Please keep the original method var name and wrap Solution* around it. Thanks.

@yaodehaung
Copy link
Contributor Author

yaodehaung commented Jul 15, 2020

@fishercoder1534
example: var twoSum = {

Solution1: function (){},
Solution2: function(){}

}

right?

@fishercoder1534
Copy link
Owner

@fishercoder1534 fishercoder1534 merged commit 0f83fb5 into fishercoder1534:master Jul 15, 2020
fishercoder1534 added a commit that referenced this pull request Jul 15, 2020
@fishercoder1534
Copy link
Owner

Solution 2 cannot pass this test case:

[3,2,4]
6

on LeetCode.

Please make sure your solutions are AC'ed on LeetCode. Thanks!

@yaodehaung
Copy link
Contributor Author

@fishercoder1534 thx

@yaodehaung yaodehaung deleted the feature/_1Solution2.js branch September 5, 2020 14:55
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