Skip to content
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.

Sign up for GitHub

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

Jump to bottom

5. Count the number of consistent strings (HASHTABLE) #241

Merged
merged 5 commits into from
Aug 5, 2021
Merged

5. Count the number of consistent strings (HASHTABLE) #241

merged 5 commits into from
Aug 5, 2021

Conversation

soma2000-lang
Copy link
Contributor

#64
code done in c++

@soma2000-lang
Copy link
Contributor Author

@Sargam-Agarwal pleasee check

@Sargam-Agarwal
Copy link
Collaborator

Hey @soma2000-lang,
Kindly fulfill all the requirements of the codebase as mentioned in the README.
Please take care of these things while contributing.

Sargam-Agarwal
Sargam-Agarwal requested changes Jul 29, 2021
View reviewed changes
Copy link
Collaborator

@Sargam-Agarwal Sargam-Agarwal left a comment

Choose a reason for hiding this comment

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

Kindly add the main function and other requirements as per the codebase.

@soma2000-lang
Copy link
Contributor Author

@Sargam-Agarwal done

@soma2000-lang
Copy link
Contributor Author

@Sargam-Agarwal please merge

@Sargam-Agarwal
Copy link
Collaborator

Hey @soma2000-lang,
Kindly add the header files as well. How come your code is running without header files !?

@soma2000-lang
Copy link
Contributor Author

@Sargam-Agarwal done

@soma2000-lang
Copy link
Contributor Author

@Sargam-Agarwal If this one gets merged can u please assign me #62

@Sargam-Agarwal
Copy link
Collaborator

Sorry @soma2000-lang, but it doesn't seem like you are doing these codes by yourself. Do you think these header files are enough to run your code correctly ?
I will see to this code only now. I wont be assigning you any issue from now on.
Find the mistake. Rectify it if possible. Ill merge this one.

@soma2000-lang
Copy link
Contributor Author

soma2000-lang commented Aug 5, 2021 •
edited
Loading

@Sargam-Agarwal I think this header file will suffice #include <bits/stdc++.h>
and sorry in a hurry I fogot to include this one

@Sargam-Agarwal
Copy link
Collaborator

@soma2000-lang,
Merging it.

@Sargam-Agarwal Sargam-Agarwal merged commit f7234fc into LetsGrowMoreCommunity:main Aug 5, 2021
@rohan-kulkarni-25 rohan-kulkarni-25 added the LGMSOC21 This issue will be considered for LGM-SOC 21 label Oct 3, 2021
@Sargam-Agarwal Sargam-Agarwal added the 🟨 Level2 This issue will be considered as level 2 issue for LGM-SOC 21.Points will be 15 label Oct 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sargam-Agarwal Sargam-Agarwal Sargam-Agarwal requested changes

Assignees

@soma2000-lang soma2000-lang

Labels
🟨 Level2 This issue will be considered as level 2 issue for LGM-SOC 21.Points will be 15 LGMSOC21 This issue will be considered for LGM-SOC 21
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@soma2000-lang @Sargam-Agarwal @rohan-kulkarni-25