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

chore: comment cleanup #372

Merged
merged 1 commit into from
Aug 23, 2023
Merged

chore: comment cleanup #372

merged 1 commit into from
Aug 23, 2023

Conversation

horsefacts
Copy link
Collaborator

@horsefacts horsefacts commented Aug 23, 2023

Motivation

Assorted small comment fixes.

Merge Checklist

Choose all relevant options below by adding an x now or at any time before submitting for review


PR-Codex overview

This PR focuses on updating the code comments in the Bundler.sol and KeyRegistry.sol files.

Detailed summary

  • Updated comment in Bundler.sol to clarify the trusted caller's role during the bootstrap phase.
  • Updated comment in KeyRegistry.sol to clarify the actions that are paused and unpaused by the owner.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@horsefacts horsefacts added the chore A task that is none of the other types label Aug 23, 2023
@github-actions
Copy link

Coverage after merging horsefacts/comment-fixes into main will be

98.65%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   Bundler.sol100%100%100%100%
   FnameResolver.sol100%100%100%100%
   IdRegistry.sol100%100%100%100%
   KeyRegistry.sol100%100%100%100%
   StorageRegistry.sol100%100%100%100%
src/lib
   EIP712.sol50%100%50%50%19
   Signatures.sol100%100%100%100%
   TransferHelper.sol0%0%0%0%12, 17, 20, 20, 20
   TrustedCaller.sol100%100%100%100%
src/validators
   SignedKeyRequestValidator.sol100%100%100%100%

@horsefacts horsefacts merged commit 5cf9b4a into main Aug 23, 2023
2 checks passed
@horsefacts horsefacts deleted the horsefacts/comment-fixes branch August 23, 2023 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore A task that is none of the other types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant