-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
feat: add Substitution Cipher algorithm and tests #1773
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
feat: add Substitution Cipher algorithm and tests #1773
Conversation
Codecov Report
All modified and coverable lines are covered by tests ✅
Project coverage is 85.91%. Comparing base (
1d252d7
) to head (25add22
).
Additional details and impacted files
@@ Coverage Diff @@ ## master #1773 +/- ## ========================================== + Coverage 85.87% 85.91% +0.04% ========================================== Files 377 378 +1 Lines 19720 19779 +59 Branches 3005 3017 +12 ========================================== + Hits 16935 16994 +59 Misses 2785 2785
☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
This PR only introduces the Substitution Cipher and its test. I’ve noticed the CI flags code style/spelling issues in files like:
String/AlphaNumericPalindrome.js
Recursive/SubsequenceRecursive.js
Maths/MobiusFunction.js
These issues are pre-existing in the repository and not introduced by this PR. I have run Prettier on my changes, and there are no further modifications I can make to fix these unrelated files.
Open in Gitpod know more
Describe your change:
Checklist:
Summary:
This PR adds a monoalphabetic Substitution Cipher to the
ciphers/
directory.substitutionCipherEncryption
andsubstitutionCipherDecryption
functions📚 Reference: Wikipedia – Substitution Cipher