Skip to content

Conversation

@aaditkamat
Copy link

Please make sure you have done the following:

  • Created a folder with the name of language you code inside /Day<n>/ directory (Where n is for nth day)
  • Added your Codes
  • Tested your codes
  • Added credits (author and date) at the top of the code
  • Updated README.md
  • Added yourself as a contributor using all-conributors-cli

Aadit Rahul Kamat and others added 30 commits December 21, 2018 22:25
# Conflicts:
#	.all-contributorsrc
#	CONTRIBUTORS.md
#	Day1/README.md
@MadhavBahl
Copy link
Member

Hey @aaditkamat looks like there are some merge conflicts,
I guess the file name that you have used has already been used, try changing it?

@MadhavBahl MadhavBahl requested a review from Razdeep December 28, 2018 13:35
@aaditkamat aaditkamat changed the title Add Ruby implementation for Day 6 problem Add Ruby implementation for Day 6 problem and Ruby, C++ & Java implementations for Day 7 problem Dec 28, 2018
@aaditkamat aaditkamat changed the title Add Ruby implementation for Day 6 problem and Ruby, C++ & Java implementations for Day 7 problem Add Ruby implementation for Day 6 problem; Add Ruby, C++ & Java implementations for Day 7 problem Dec 28, 2018
@aaditkamat aaditkamat changed the title Add Ruby implementation for Day 6 problem; Add Ruby, C++ & Java implementations for Day 7 problem Add Ruby implementation for Day 6 problem; Add Ruby, C++ & Java implementations for Day 7 problem Dec 28, 2018
@aaditkamat aaditkamat changed the title Add Ruby implementation for Day 6 problem; Add Ruby, C++ & Java implementations for Day 7 problem Add Ruby implementation for Day 6 problem; Add Ruby, C++ & Java implementations for Day 7 problem Dec 28, 2018
@aaditkamat
Copy link
Author

aaditkamat commented Dec 28, 2018

Sorry for these extraneous change in titles, I was trying to format the title properly

Copy link
Member

@MadhavBahl MadhavBahl left a comment

Choose a reason for hiding this comment

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

Please check the README, some codes were removed,

it might be a case that someone's PR was accepted and after that you pushed your changes, I guess due to that some lines were removed.

Please fix it.

* @author : dhruv-gupta14
* @date : 27/12/2018
*/
```ruby
Copy link
Member

Choose a reason for hiding this comment

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

Where are the @author and @date credits

print("Reverse of string: ", reverse_words(string)})
```

## C++ Implementation
Copy link
Member

Choose a reason for hiding this comment

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

Why did you remove the C++ implementation

puts "\nAre #{first_str} and #{second_str} anagrams? #{check_anagram(String.new(first_str), String.new(second_str))}"
```

### [Anagram Check by @imkaka](./C++/checkAnagram.cpp)
Copy link
Member

Choose a reason for hiding this comment

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

Why was this removed?

print("Reverse of string \'", string, "\': ", reverse_words(string), sep='')
```

### [Word Reverse by @imkaka](./C++/wordReverse.cpp)
Copy link
Member

Choose a reason for hiding this comment

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

Why was this removed?

@MadhavBahl MadhavBahl merged commit 3afd69e into CodeToExpress:master Dec 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants