-
-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
feat: Add Travelling Salesman Problem(Naive Approach) #1590
Conversation
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.
Hey, Thanks for your contribution. We request you to follow contribution guidelines
Made some changes. |
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.
Please follow contribution guidelines
Thanks.
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.
Thanks for reviewing, @ayaankhan98! 😄
@Mayank17M, in case you are confused or don't know what to do, you can check the typical structure of a program. I hope that helps. Let us know if you need any more help. 🙂
Thankyou @Panquesito7 , I'll make the necessary changes. |
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.
Great work! 😄👍
Co-authored-by: David Leal <[email protected]>
I think all the requested changes have been covered. |
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.
Almost there! Great work! 😄👍
Please enable GitHub Actions in your repository of this fork in this link: https://github.com/Mayank17M/C-Plus-Plus/actions
Co-authored-by: David Leal <[email protected]>
Co-authored-by: David Leal <[email protected]>
Co-authored-by: David Leal <[email protected]>
Co-authored-by: David Leal <[email protected]>
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.
This is all that needs to be addressed, and your PR will be merged! 🎉
Co-authored-by: David Leal <[email protected]>
fa139f8
to
9b9b4a9
Compare
Addressed all the requested changes. Please review. Thanks! |
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.
Awesome work, @Mayank17M! It's been great for your first contribution here! We hope you keep contributing! 😄👍🎉
@Panquesito7: Here's the output, goes in: Is it shows correct? |
That looks correct for me. @Mayank17M, does that look correct? |
Yep, that's correct :) |
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.
@Mayank17M: LGTM! 🥳
Thanks for reviewing @Amino19! 😄 |
Description of Change
Checklist
Notes: