-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
New Hashmap functionality added #42
Conversation
Hey there! 👋 This repository uses the Prettier code style. You forgot to format these files in your pull request:
The Prettier installation guide is a good place to get started with formatting your code properly. ❤️ Your friendly Prettifier bot. |
index = randomInt(0, array.length); | ||
} | ||
viewed[index] = 1; | ||
return array[index].line; |
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.
So, here it will check if the line is previously generated or not, and if it is not generated then it will return the line?
It will be great if you can provide a short description of what your code exactly does
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.
Yeah ohk i will just desribe those
hey @AnirudhPanda I have added those descriptions you asked for hope this time my PR gets merged , thank you |
Hey, @gitaalekhyapaul would you mind testing this? I guess this is done only for the |
@AnirudhPanda you can go ahead and merge the new functionalities if you like, I'll add tests for them in #36 |
|
I guess for that we need to then add hashmap for every type of quote like for inspire and cheerful etc ... |
Currently we have the |
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.
Can you update the changes in Readme for the users and the devs!!
Planning to launch this in v1.2.0
* Update data.json * Update data.json Co-authored-by: Anirudh Panda <[email protected]>
Hey @AnirudhPanda I made those changes you requested in readme file but unable to push on v1.2.0 |
Hey @AnirudhPanda I have created new Hashmap issue and also PR please have a look .