Skip to content

/ Javascript

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

Jump to bottom

Added Tree Sort algorithms #513

Open
wants to merge 1 commit into
base: master
from
Open

Added Tree Sort algorithms #513

wants to merge 1 commit into from

Conversation

@sonechca
Copy link

@sonechca sonechca commented Oct 25, 2020 •
edited

Welcome to JavaScript community

Describe your change:

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new JavaScript files are placed inside an existing directory.
  • All filenames should use the UpperCamelCase (PascalCase) style. There should be no spaces in filenames.
    Example:UserProfile.js is allowed but userprofile.js,Userprofile.js,user-Profile.js,userProfile.js are not
  • All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
  • If this pull request resolves one or more open issues then the commit message contains Fixes: #{$ISSUE_NO}.
@sonechca
Added Tree Sort algorithms
Loading status checks…
1d31b58
@sonechca sonechca mentioned this pull request Oct 25, 2020
Open
@ruppysuppy
ruppysuppy requested changes Oct 25, 2020
View changes
Copy link
Member

@ruppysuppy ruppysuppy left a comment

Make sure that the checks pass, its most probably failing due to improper styling. Use standard as mentioned in CONTRIBUTING.md
@cclauss
Copy link
Member

@cclauss cclauss commented Oct 31, 2020

Run standard --fix to automatically fix some problems.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ruppysuppy ruppysuppy

Assignees
No one assigned
Labels
Beginner failing npm standards good first issue
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@sonechca @cclauss @ruppysuppy @itsvinayak
You can’t perform that action at this time.