Skip to content

Conversation

itsmohitmkk
Copy link
Contributor

@itsmohitmkk itsmohitmkk commented Nov 27, 2021

Fixes : #795
Implementation of Fenwick Tree.
Link for Description : https://www.geeksforgeeks.org/binary-indexed-tree-or-fenwick-tree-2/

@raklaptudirm
Copy link
Member

Please read the contribution guidelines and change your code accordingly.

@itsmohitmkk
Copy link
Contributor Author

Read the contribution guidelines. Sorry, I didn't follow it, I am new to open source. I made a few corrections to the code, please check if it's okay now.

Copy link
Member

@raklaptudirm raklaptudirm left a comment

Choose a reason for hiding this comment

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

Format your code with standard.js.

Copy link
Member

@raklaptudirm raklaptudirm left a comment

Choose a reason for hiding this comment

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

Remove "implementation" from the file names.

@itsmohitmkk
Copy link
Contributor Author

Remove "implementation" from the file names.

Changed name of files
Changed imports.

@raklaptudirm raklaptudirm added algorithm Adds or improves an algorithm feature Adds a new feature Reviewed labels Nov 30, 2021
@raklaptudirm raklaptudirm merged commit c3b2bac into TheAlgorithms:master Nov 30, 2021
@itsmohitmkk itsmohitmkk deleted the FenwickTreeImplementation branch November 30, 2021 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
algorithm Adds or improves an algorithm feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fenwick Tree Implementation
2 participants