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.

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

Closed

Conversation

DADDY-DOUBLESHOT
Copy link

Hey @AnirudhPanda I have created new Hashmap issue and also PR please have a look .

@ghost
Copy link

ghost commented Oct 4, 2021

Hey there! 👋 This repository uses the Prettier code style.

You forgot to format these files in your pull request:

  • index.js

The Prettier installation guide is a good place to get started with formatting your code properly.
Thanks!

❤️

Your friendly Prettifier bot.

@AnirudhPanda AnirudhPanda added the hacktoberfest Hacktoberfest 2021 label Oct 4, 2021
index = randomInt(0, array.length);
}
viewed[index] = 1;
return array[index].line;
Copy link
Member

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

Copy link
Author

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

@DADDY-DOUBLESHOT
Copy link
Author

hey @AnirudhPanda I have added those descriptions you asked for hope this time my PR gets merged , thank you

@AnirudhPanda
Copy link
Member

AnirudhPanda commented Oct 4, 2021

Hey, @gitaalekhyapaul would you mind testing this?
Shall I merge it to v1.1.0 and then you'll test?

I guess this is done only for the array and not for the isnpireArray ?

@AnirudhPanda AnirudhPanda changed the base branch from main to v1.1.0 October 4, 2021 04:55
@gitaalekhyapaul
Copy link
Contributor

@AnirudhPanda you can go ahead and merge the new functionalities if you like, I'll add tests for them in #36

@AnirudhPanda AnirudhPanda added the help wanted Extra attention is needed label Oct 5, 2021
@AnirudhPanda
Copy link
Member

Hey, @gitaalekhyapaul would you mind testing this? Shall I merge it to v1.1.0 and then you'll test?

I guess this is done only for the array and not for the isnpireArray ?

@DADDY-DOUBLESHOT

@DADDY-DOUBLESHOT
Copy link
Author

I guess for that we need to then add hashmap for every type of quote like for inspire and cheerful etc ...
@AnirudhPanda

@AnirudhPanda
Copy link
Member

I guess for that we need to then add hashmap for every type of quote like for inspire and cheerful etc ... @AnirudhPanda

Currently we have the inspire array as an extra

@AnirudhPanda AnirudhPanda changed the base branch from v1.1.0 to v1.2.0 October 10, 2021 10:18
Copy link
Member

@AnirudhPanda AnirudhPanda left a 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

dikshantgautam2k and others added 3 commits October 10, 2021 18:40
* Update data.json

* Update data.json

Co-authored-by: Anirudh Panda <[email protected]>
@github-actions github-actions bot removed the hacktoberfest Hacktoberfest 2021 label Oct 11, 2021
@github-actions github-actions bot added data Changes made in data folder hacktoberfest Hacktoberfest 2021 labels Oct 11, 2021
@DADDY-DOUBLESHOT
Copy link
Author

Hey @AnirudhPanda I made those changes you requested in readme file but unable to push on v1.2.0
Can you guide me here a little bit about on which branch shall I push those changes and how...
thank you

@AnirudhPanda AnirudhPanda deleted the branch Opentek-Org:v1.2.0 November 3, 2021 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data Changes made in data folder hacktoberfest Hacktoberfest 2021 help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hashmap Function to generate new Lines after each retrieval from same client
4 participants