-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Improving the coding standard for AvLTree Data Structure #882
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
Improving the coding standard for AvLTree Data Structure #882
Conversation
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.
Please create a /test sub-directory and write some jest tests for the algorithm there. I know they were not there, but they will further enhance the code quality and help catch bugs.
~ Created test cases for AVL Tree ~ Indentation fix for AVLTree.js
Please create a
/testsub-directory and write somejesttests for the algorithm there. I know they were not there, but they will further enhance the code quality and help catch bugs.
The test cases are created
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.
The tests are failing, so please check the error and fix them.
...mproving-coding-standard-of-AVLTree-Data-structure
The tests are failing, so please check the error and fix them.
Test cases are working properly in local without any issues
Please check if you have any local changes which you have not yet committed.
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.
Please format your code with standard.js.
Welcome to JavaScript community
Open in Gitpod know more
Describe your change:
Checklist:
Example:
UserProfile.jsis allowed butuserprofile.js,Userprofile.js,user-Profile.js,userProfile.jsare notFixes: #{$ISSUE_NO}.