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

Add Morse Decoder. #212

Merged
merged 3 commits into from
Aug 29, 2021
Merged

Add Morse Decoder. #212

merged 3 commits into from
Aug 29, 2021

Conversation

aalekhpatel07
Copy link
Contributor

Feature

Add a function to decode a given string of Morse Code.

Signed-off-by: Aalekh Patel aalekh.gwpeck.7998@icloud.com

Signed-off-by: Aalekh Patel <aalekh.gwpeck.7998@icloud.com>
@imp2002
Copy link
Member

imp2002 commented Aug 28, 2021

thank you, but the CI has failed, please run cargo fmt --all -- --check, cargo clippy --all -- -D warnings, cargo test

…s to something more intuitive.

Signed-off-by: Aalekh Patel <aalekh.gwpeck.7998@icloud.com>
@aalekhpatel07
Copy link
Contributor Author

aalekhpatel07 commented Aug 28, 2021

There! Clippy is happy now ;)

@imp2002
Copy link
Member

imp2002 commented Aug 28, 2021

thank you, i will review it a few hours later, have something to do.

siriak
siriak previously approved these changes Aug 28, 2021
imp2002
imp2002 previously approved these changes Aug 28, 2021
@imp2002
Copy link
Member

imp2002 commented Aug 28, 2021

hello, can help update the README?

Signed-off-by: Aalekh Patel <aalekh.gwpeck.7998@icloud.com>
@aalekhpatel07 aalekhpatel07 dismissed stale reviews from imp2002 and siriak via b01d379 August 28, 2021 17:26
@aalekhpatel07
Copy link
Contributor Author

Readme has been updated.

@imp2002 imp2002 merged commit b84e503 into TheAlgorithms:master Aug 29, 2021
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.

3 participants