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

Tree_Preorder_Traversal.js Added #2756

Closed
wants to merge 54 commits into from
Closed

Conversation

afroz23
Copy link
Contributor

@afroz23 afroz23 commented Apr 14, 2020

Fixes #2550

Checklist:

  • 4 space indentation.
  • Coding conventions are followed.
  • Input is taken dynamically.
  • Sample Input / Output is added at the end of file.
  • Logic Documentation (Comments).
  • File names are correct.

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

Changes proposed in this pull request:

  • Tree_Preorder_Traversal.js Added

Languages Used:

  • JavaScript

Files Added:

  • Tree_Preorder_Traversal.js

We're happy to help you get this ready -- don't be afraid to ask for help.

Thanks!

@afroz23
Copy link
Contributor Author

afroz23 commented Apr 20, 2020

@singh-shreya6 mam review it :)

singh-shreya6
singh-shreya6 previously approved these changes Apr 24, 2020
@singh-shreya6
Copy link
Collaborator

Squash your commits @afroz23. The code looks good.

afroz23 and others added 9 commits April 25, 2020 12:18
Priority_Queue
* Create Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs

* Update Pancake_Sort.cs
* Create Tree_Preorder_Traversal.kt

* Update Tree_Preorder_Traversal.kt

Taking dynamic input
* add kosaraju algorithm in cpp

* Update kosaraju_algorithm.cpp
* Create GENERIC_TREE_LOTRAVERSAL.java

* Update GENERIC_TREE_LOTRAVERSAL.java

* Revert "Update GENERIC_TREE_LOTRAVERSAL.java"

This reverts commit dfdd309.

* Revert "Create GENERIC_TREE_LOTRAVERSAL.java"

This reverts commit 2c87bbb.

* Create Aggressive_Cows.java

* Update Aggressive_Cows.java

1. Multi comments made
2. Extra spaces and lines removed
3. Indentation corrected
AkanshaKamboj and others added 20 commits April 25, 2020 12:27
…aman224#2395)

* Create Linked_List_Merge_Sort.c

* Add files via upload

* Update Linked_List_Merge_Sort.c

* Update Linked_List_Merge_Sort.c

* Update Linked_List_MergeSort.cpp

* Update Linked_List_MergeSort.java

* Update Linked_List_MergeSort.java

* Update Linked_List_MergeSort.py

* Update Linked_List_MergeSort.cpp

* Update Linked_List_MergeSort.java

* Update Linked_List_Merge_Sort.c

* Rename Linked_List_MergeSort.cpp to Linked_List_Merge_Sort.cpp

* Rename Linked_List_MergeSort.py to Linked_List_Merge_Sort.py

* Rename Linked_List_MergeSort.java to Linked_List_Merge_Sort.java
* Chinese Remainder Theorem in Go

* Added comments
* Create Kadane Algorithm in GO
Used disjoint set data structure for Algorithm implementation.
This reverts commit 8d5c048.
Copy link
Collaborator

@singh-shreya6 singh-shreya6 left a comment

Choose a reason for hiding this comment

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

Seems like your squashing has gone wrong. The files changed should be one. Also the number of commits in the second tab here should show as 1 after squashing. Now it shows 51. Please fix it.

Tree_Preorder_Traversal/Tree_Preorder_Traversal.js.js Outdated Show resolved Hide resolved
@singh-shreya6
Copy link
Collaborator

You are not squashing your commits, as you can see the number of commits have increased to 54, rather than decreasing to 1. If you are having trouble squashing, reach out to someone on slack or open a new PR

@afroz23 afroz23 closed this Apr 27, 2020
@afroz23 afroz23 reopened this Apr 27, 2020
@afroz23 afroz23 closed this Apr 27, 2020
@afroz23 afroz23 reopened this Apr 27, 2020
@afroz23 afroz23 closed this Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tree Preorder Traversal