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 railfence cipher #715

Merged
merged 4 commits into from
Apr 24, 2024

Conversation

Chandrahas77
Copy link
Contributor

Added rail fence cipher functionality inform of encrypt and decrypt functions to cipher package.

@Chandrahas77 Chandrahas77 force-pushed the add-railfence-cipher branch from 2867678 to 0a9b3a9 Compare April 18, 2024 04:18
@Chandrahas77
Copy link
Contributor Author

@raklaptudirm Can you please have a look at this PR?

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.36%. Comparing base (0d0b97a) to head (e07da8b).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #715      +/-   ##
==========================================
+ Coverage   87.25%   87.36%   +0.10%     
==========================================
  Files         199      200       +1     
  Lines        5274     5317      +43     
==========================================
+ Hits         4602     4645      +43     
  Misses        533      533              
  Partials      139      139              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@raklaptudirm raklaptudirm merged commit c5173f3 into TheAlgorithms:master Apr 24, 2024
5 checks passed
sozelfist pushed a commit to sozelfist/Go that referenced this pull request Apr 29, 2024
* added rail fence cipher implementation

* updated readme

* fixed typo

---------

Co-authored-by: Rak Laptudirm <[email protected]>
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