-
Notifications
You must be signed in to change notification settings - Fork 90
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
Submit a Leetcode problem #49
Comments
i would like to work on this issue. Please assing it to me |
Hey! Thanks for showing interest! This issue is generic. Feel free to grab any problem that is not in this repo and work on it. You can also submit an issue while you're tackling the problem. |
hey, I would like to contribute my leetcode problems solutions in this repo! |
Hey thank you!
Sure, the PR needs to have a test file that tests a few scenarios. The file should have a main function See example: const assert = require('assert');
const { theNewProblem } = require('../LeetcodeProblems/The_New_Problem.js');
function test() {
// Test a few scenarios here like this:
assert.strictEqual(theNewProblem, [.....]);
}
module.exports.test = test |
@deepesh85b you are welcome to contribute. No need to be assigned to this issue to work on it :). Thank you! |
Hey ! |
@kav1239 you are welcome to contribute. No need to be assigned to this issue to work on it :). Thank you! |
Hi dear, can u assign me and please add the October label to confirm and start my contribution |
Is the issue open?I can help contribute.. |
yes, it is open. Thanks |
hello, can u assign me this issue and please add the hacktoberfest label to confirm and start my contribution |
Pick a problem in Leetcode or any other platform.
If you find a solution, follow the next steps:
Open A PR that contains:
test()
function which will run all the tests of the file.PR Example: #39
If you don't find a solution but want to discuss a problem, I would be happy to! In that case, please submit an issue.
The text was updated successfully, but these errors were encountered: