Skip to content
This repository was archived by the owner on Sep 22, 2021. It is now read-only.

Conversation

@GiorgiBeriashvili
Copy link

IMPORTANT: Please do not create a Pull Request without an issue.

All Submissions:

  • I have read the CONTRIBUTING document.
  • My code is written in Python3 and is ending with .py
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • I have checked that my submission does pass the test on LeetCode.com
  • Does your filename follow the naming Conventions?
  • Have you linked your PR to an Issue?

Closing issues

Resolves the following issue: #264

Copy link
Owner

@vJechsmayr vJechsmayr 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 your Algorithm once more - there is a Problem with one of the Testcases on Leetcode:
image

https://leetcode.com/problems/ransom-note/submissions/

Copy link
Owner

@vJechsmayr vJechsmayr left a comment

Choose a reason for hiding this comment

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

see comment above

@GiorgiBeriashvili
Copy link
Author

Curious... I will make adjustments right away!

@GiorgiBeriashvili
Copy link
Author

I changed the faulty solution in order to work on the failed testcase and maybe improved its tangibility.

I also hopefully didn't mess anything up by committing again, it was my first time doing something like this. 😅

Copy link
Owner

@vJechsmayr vJechsmayr left a comment

Choose a reason for hiding this comment

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

Looks good now! 😸 Thanks for contributing - Merged!

Runtime: 36 ms

Happy Hacktoberfest!

@vJechsmayr vJechsmayr merged commit e63154a into vJechsmayr:master Oct 4, 2020
@GiorgiBeriashvili GiorgiBeriashvili deleted the feature/ransom-note branch October 4, 2020 20:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants