Merge branch 'master' into master

Author: kilavvy

README.md

@@ -91,9 +91,9 @@ contract TestContract is Test {
         assertEq(test.exists(), 100);
     }
 
-    // It supports arbitrary storage layouts, like assembly based storage locations
+    // It supports arbitrary storage layouts, like assembly-based storage locations
     function testFindHidden() public {
-        // `hidden` is a random hash of a bytes, iteration through slots would
+        // `hidden` is a random hash of bytes; iterating through slots would
         // not find it. Our mechanism does
         // Also, you can use the selector instead of a string
         uint256 slot = stdstore.target(address(test)).sig(test.hidden.selector).find();
@@ -167,7 +167,7 @@ contract Storage {
 
 ### stdCheats
 
-This is a wrapper over miscellaneous cheatcodes that need wrappers to be more dev friendly. Currently there are only functions related to `prank`. In general, users may expect ETH to be put into an address on `prank`, but this is not the case for safety reasons. Explicitly this `hoax` function should only be used for addresses that have expected balances as it will get overwritten. If an address already has ETH, you should just use `prank`. If you want to change that balance explicitly, just use `deal`. If you want to do both, `hoax` is also right for you.
+This is a wrapper around miscellaneous cheatcodes that need wrappers to be more dev-friendly. Currently there are only functions related to `prank`. In general, users may expect ETH to be put into an address on `prank`, but this is not the case for safety reasons. Explicitly, this `hoax` function should only be used for addresses that have expected balances as it will get overwritten. If an address already has ETH, you should just use `prank`. If you want to change that balance explicitly, just use `deal`. If you want to do both, `hoax` is also right for you.
 
 #### Example usage:
 

RELEASE_CHECKLIST.md

@@ -8,5 +8,5 @@ This checklist is meant to be used as a guide for the `forge-std` release proces
 - [ ] Open and merge a PR with the version bump
 - [ ] Tag the merged commit with the version number: `git tag v<X.Y.Z>`
 - [ ] Push the tag to the repository: `git push --tags`
-- [ ] Create a new GitHub release with the automatically generated changelog and with the name set to `v<X.Y.Z>`
+- [ ] Create a new GitHub release with the automatically generated changelog and the name set to `v<X.Y.Z>`
 - [ ] Add `## Featured Changes` section to the top of the release notes

src/StdToml.sol

@@ -8,16 +8,16 @@ import {VmSafe} from "./Vm.sol";
 // ```
 // using stdToml for string;
 // string memory toml = vm.readFile("<some_path>");
-// toml.readUint("<json_path>");
+// toml.readUint("<toml_path>");
 // ```
 // To write:
 // ```
 // using stdToml for string;
-// string memory json = "json";
-// json.serialize("a", uint256(123));
-// string memory semiFinal = json.serialize("b", string("test"));
-// string memory finalJson = json.serialize("c", semiFinal);
-// finalJson.write("<some_path>");
+// string memory toml = "toml";
+// toml.serialize("a", uint256(123));
+// string memory semiFinal = toml.serialize("b", string("test"));
+// string memory finalToml = toml.serialize("c", semiFinal);
+// finalToml.write("<some_path>");
 // ```
 
 library stdToml {

src/Vm.sol

@@ -44,7 +44,7 @@ interface IERC7540Deposit is IERC7540Operator {
      * @dev Transfers assets from sender into the Vault and submits a Request for asynchronous deposit.
      *
      * - MUST support ERC-20 approve / transferFrom on asset as a deposit Request flow.
-     * - MUST revert if all of assets cannot be requested for deposit.
+     * - MUST revert if all assets cannot be requested for deposit.
      * - owner MUST be msg.sender unless some unspecified explicit approval is given by the caller,
      *    approval of ERC-20 tokens from owner to sender is NOT enough.
      *
@@ -107,7 +107,7 @@ interface IERC7540Redeem is IERC7540Operator {
      *
      * - MUST support a redeem Request flow where the control of shares is taken from sender directly
      *   where msg.sender has ERC-20 approval over the shares of owner.
-     * - MUST revert if all of shares cannot be requested for redeem.
+     * - MUST revert if all shares cannot be requested for redeem.
      *
      * @param shares the amount of shares to be redeemed to transfer from owner
      * @param controller the controller of the request who will be able to operate the request