Skip to content

Added Solution for a problem 244 #3538

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

Closed
wants to merge 3 commits into from

Conversation

pallasivasai
Copy link
Contributor

@pallasivasai pallasivasai commented Jul 18, 2024

Related Issue

Fix #3530

Description

Type of PR

  • Feature enhancement
  • Documentation update

Screenshots / Videos (if applicable)

Checklist

  • I have performed a self-review of my code.
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have followed the code style guidelines of this project.
  • I have checked for any existing open issues that my pull request may address.
  • I have ensured that my changes do not break any existing functionality.
  • Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
  • I have read the resources for guidance listed below.
  • I have followed security best practices in my code changes.

@pallasivasai pallasivasai changed the title added Solution for a problem 244 Added Solution for a problem 244 Jul 18, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, @pallasivasai! 🎉 Thank you for submitting your pull request to CodeHarborHub. We appreciate your contribution and enthusiasm! Our team will review it soon. If you have any questions or need further assistance, feel free to reach out. Thanks for contributing!

Copy link
Contributor

deepsource-io bot commented Jul 18, 2024

Here's the code health analysis summary for commits fcae68b..db43972. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Copy link

github-actions bot commented Jul 18, 2024

⚡️ Lighthouse Report for the Deploy Preview of this PR 🚀

🔗 Site: CodeHarborHub | Live Site

URL 🌐 Performance Accessibility Best Practices SEO 📊
/ 🔴 27 🟡 86 🟡 75 🟡 86 📄
/docs 🔴 42 🟢 96 🟢 96 🟢 100 📄
/docs/category/javascript 🔴 42 🟢 96 🟢 96 🟢 100 📄
/courses 🔴 46 🟢 96 🟢 96 🟢 100 📄
/courses/category/reactjs 🔴 48 🟢 96 🟢 96 🟢 100 📄
/dsa-problems 🔴 48 🟢 96 🟢 96 🟢 100 📄
/dsa-problems/category/leetcode-problems 🔴 46 🟢 96 🟢 96 🟢 100 📄
/dsa-solutions 🟡 50 🟢 96 🟢 96 🟢 100 📄
/dsa-solutions/category/leetcode-solutions 🔴 44 🟢 96 🟢 96 🟢 100 📄
/blog 🔴 34 🟢 95 🟢 96 🟢 100 📄
/showcase 🟡 50 🟢 95 🟢 96 🟡 86 📄
/community 🟡 55 🟢 96 🟢 96 🟢 100 📄
/docs/tags 🔴 46 🟢 96 🟢 96 🟡 86 📄
/product 🟡 52 🟢 96 🟢 96 🟢 100 📄

@ajay-dhangar ajay-dhangar added GSSOC'24 GirlScript Summer of Code | Contributor level1 GirlScript Summer of Code | Contributor's Levels gssoc GirlScript Summer of Code | Contributor dsa-solution Label for solutions related to Data Structures & Algorithms. labels Jul 18, 2024
Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete the 0185-Department-Top-Three-Salaries.md file because it is a duplicate. This file is already available in this repository.

@pallasivasai
Copy link
Contributor Author

Okay Done please check now @ajay-dhangar

Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code doesn't complete the problem.

@pallasivasai
Copy link
Contributor Author

can you explain it more detail but this code is enough!

@ajay-dhangar
Copy link
Member

can you explain it more detail but this code is enough!

@pallasivasai, please complete your code as it is currently unfinished. Thank you!

Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pallasivasai, we need more content after line no. 244 because your code is incomplete. The function "shortest" is not defined, so the code will not run. Please provide the missing code or a link to the completed gist.

@sivaprasath2004
Copy link
Contributor

Hello @ajay-dhangar can i solve this PR..

@ajay-dhangar
Copy link
Member

Hello @ajay-dhangar can i solve this PR..

@sivaprasath2004, Okay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dsa-solution Label for solutions related to Data Structures & Algorithms. gssoc GirlScript Summer of Code | Contributor GSSOC'24 GirlScript Summer of Code | Contributor level1 GirlScript Summer of Code | Contributor's Levels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: want to add Add Solution for a problem 244 (Premium)
3 participants