fix(cipher): resolve unclosed block loop syntax error in aesInstrumented - #1853
Closed
Chigael wants to merge 1 commit into
Closed
fix(cipher): resolve unclosed block loop syntax error in aesInstrumented#1853Chigael wants to merge 1 commit into
Chigael wants to merge 1 commit into
Conversation
|
@Chigael is attempting to deploy a commit to the csxark's projects Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
🎉 Thank You for Your ContributionHello @Chigael, Thank you for submitting a Pull Request to CryptoViz. We appreciate the time and effort you've invested in contributing to the project. Your Pull Request has been received successfully and will be reviewed by the maintainers as soon as possible. 📋 Pull Request Checklist
Ensuring these requirements are met helps streamline the review process and enables maintainers to review your contribution more efficiently. ❤️ Support CryptoVizIf you find CryptoViz helpful, consider supporting the project by:
Your support helps increase the project's visibility and encourages continued development. Thank you for being a part of the CryptoViz community! Thank you once again for contributing to CryptoViz. We appreciate your support and look forward to reviewing your contribution. |
Contributor
|
Important Review skippedToo many files! This PR contains 514 files, which is 414 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (514)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Owner
|
Hi @Chigael, this PR cannot be merged as the Program was already over by the time you raised it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Description
Fixes a syntax error in lib/cipher/symmetric/aes.ts where the for (let b = 0; b < numBlocks; b++) loop inside aesInstrumented was unclosed prior to the function's final return statement.
Related Issue
Closes #1695
Scope
This PR touches exactly one of the following. If it touches more than
one, split it into separate PRs — multi-scope PRs are the pattern that
introduces inconsistent architecture and will not be reviewed as-is.
lib/cipher/**)app/**,components/**)lib/workers/**,hooks/use*Worker.ts)docs/**,*.md, MDX content)tests/**)Changes Made
Testing
npm testpasses locally.npm run typecheckpasses locally.npm run lintpasses locally.lib/cipher/**file stays at or above 80%.A PR that adds functionality without a corresponding test in the same PR will not be merged. "Will add tests later" is not accepted.
Simulation vs. Live Data
Screenshots
If applicable, attach screenshots or screen recordings.
Checklist
Applicable Checklist
Complete the checklist matching the scope above, then paste the completed
checklist (or a link to it) here:
Architecture Review Checklist
Before adding another domain suite or feature suite, please identify and document the following to prevent code duplication:
Additional Notes
Add any additional information for reviewers here.