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

Added problem number: 229 Majority Element II in C++. #503

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

Open
Account1Adarsh wants to merge 4 commits into codedecks-in:master
base: master
Choose a base branch
Loading
from Account1Adarsh:master

Conversation

Copy link

@Account1Adarsh Account1Adarsh commented Mar 21, 2025

Pull Request Template

Description

Please include a summary of the problem and about the approach to solve the solution. Please also include relevant motivation and context.
List any dependencies that are required for this change.

Put check marks:

Have you made changes in README file ?

  • [ ✅] Added problem & solution under correct topic.
  • [ ✅] Specified Space & Time complexity.
  • [ ✅] Specified difficulty level, tag & Note(if any).

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

  • [✅ ] Test A
  • [ ✅] Test B

Make sure all below guidelines are followed else PR will get Reject:

  • [ ✅] My code follows the style guidelines of this project
  • [ ✅] I have performed a self-review of my own code
  • [ ✅] I have commented my code so that it is easy to understand
  • [ ✅] I have made corresponding changes to the documentation
  • [ ✅] My changes generate no new warnings
  • [ ✅] Any dependent changes have been merged and published in downstream modules

Copy link

welcome bot commented Mar 21, 2025

I can tell this is your first pull request! Thank you I'm so honored. 🎉🎉🎉 I'll take a look at it ASAP!

Copy link
Author

Please review it.

Copy link
Member

This is not the efficient solution for the problem. Also please check other pull requests for the references on how to add the proper solutions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@amrit-GH23 amrit-GH23 amrit-GH23 left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /