Cosmos: Duplicate codes for linked list

Created on 1 Feb 2019  Â·  21Comments  Â·  Source: OpenGenus/cosmos

there is data_structures/src/list/singly_linked_list, but there is also a code for same problem in data_structures/linked_list/
One of them ought to be removed.

bug good first issue

Most helpful comment

Yeah, I have already solved it. This issue has been solved.

On Wed, Apr 1, 2020 at 9:11 AM Prasanna Venkatesh notifications@github.com
wrote:

Looks like there is no duplicate folder in data_structures/ for linked
list.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenGenus/cosmos/issues/3934#issuecomment-607009586,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMLBWJECFJYLAY62BZPXKLTRKKZWPANCNFSM4GT2YL5Q
.

All 21 comments

Looks like this has been fixed

@rucha174 Sorry, I had written wrong file path. Both directories still exist in repo.

Is the filename correct?

@FR4NKESTI3N @rucha174 If either of you would like to resolve this issue, then by all means go for it! Make sure to reference this issue however.

I have sent a PR. Please review it.

Why is this issue still open ? @arnavb @rucha174

@code-blooded The old directory isn't completely gone yet. We moved from one file to removing the whole directory, which is pretty much a duplicate of the other one.

@arnavb Still same code of linked list is seen in both the paths.
Is this issue still open?

@arnavb Hello! Should I make the required changes in the code?

under GSSoc i would like to contribute @AdiChat

I want to contribute in it..I think I would be able to remove the duplicate directory.

@shreeya13garg / @nitashadhingra One of you can create a PR and fix this.

I have removed the singly linked list from data_structures/src/list/singly_linked_list to remove duplicates. Should I push it on Github?

I have cloned the cosmos repository and committed the changes i.e removing duplicate files now I need permission to push it on github

My PR in which I have added the modified code-:https://github.com/shreeya13garg/cosmos_PR
If there is any error let me know.

@shreeya13garg That isn't how you create a pull request. I think you should follow this guide and become proficient with git.

Can I take up this issue if it is still open ? @arnavb

@hanzohasashi33 Sure. All you have to do is delete the extra folder, basically.

Looks like there is no duplicate folder in data_structures/ for linked list.

Yeah, I have already solved it. This issue has been solved.

On Wed, Apr 1, 2020 at 9:11 AM Prasanna Venkatesh notifications@github.com
wrote:

Looks like there is no duplicate folder in data_structures/ for linked
list.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenGenus/cosmos/issues/3934#issuecomment-607009586,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMLBWJECFJYLAY62BZPXKLTRKKZWPANCNFSM4GT2YL5Q
.

@arnavb, According to @shreeya13garg, this issue has been resolved, kindly close this issue. It will help others to focus on other pending issues. :smiley:

Was this page helpful?
0 / 5 - 0 ratings

Related issues

arnavb picture arnavb  Â·  3Comments

arthurlacoste picture arthurlacoste  Â·  3Comments

ivanrein picture ivanrein  Â·  4Comments

ruchirchauhan picture ruchirchauhan  Â·  4Comments

arnavb picture arnavb  Â·  3Comments