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: fix some comments #2248

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion contracts/Depot.sol
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ contract Depot is Owned, Pausable, ReentrancyGuard, MixinResolver, IDepot {

/* ========== STATE VARIABLES ========== */

// Address where the ether and Synths raised for selling SNX is transfered to
// Address where the ether and Synths raised for selling SNX is transferred to
// Any ether raised for selling Synths gets sent back to whoever deposited the Synths,
// and doesn't have anything to do with this address.
address payable public fundsWallet;
Expand Down
2 changes: 1 addition & 1 deletion contracts/Issuer.sol
Original file line number Diff line number Diff line change
Expand Up @@ -846,7 +846,7 @@ contract Issuer is Owned, MixinSystemSettings, IIssuer {
uint rewardsSum
) internal view returns (uint totalRedeemed, uint escrowToLiquidate) {
// The balanceOf here can be considered "transferable" since it's not escrowed,
// and it is the only SNX that can potentially be transfered if unstaked.
// and it is the only SNX that can potentially be transferred if unstaked.
uint transferable = _snxBalanceOf(account);
if (redeemTarget.add(rewardsSum) <= transferable) {
// transferable is enough
Expand Down
2 changes: 1 addition & 1 deletion contracts/ProxyPerpsV2.sol
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ contract ProxyPerpsV2 is Owned {
// Accepting a for loop since implementations for a market is going to be a very limited number (initially only 2)
for (uint idx = 0; idx < _routedTargets.length; idx++) {
if (_routedTargets[idx] == implementation) {
// remove it by bringing the last one to that position and poping the latest item (if it's the latest one will do an unecessary write)
// remove it by bringing the last one to that position and poping the latest item (if it's the latest one will do an unnecessary write)
_routedTargets[idx] = _routedTargets[_routedTargets.length - 1];
_routedTargets.pop();
break;
Expand Down
2 changes: 1 addition & 1 deletion contracts/RewardEscrowV2Frozen/IRewardEscrowV2Frozen.sol
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ interface IRewardEscrowV2Frozen {
VestingEntries.VestingEntry[] calldata vestingEntries
) external;

// Return amount of SNX transfered to SynthetixBridgeToOptimism deposit contract
// Return amount of SNX transferred to SynthetixBridgeToOptimism deposit contract
function burnForMigration(address account, uint256[] calldata entryIDs)
external
returns (uint256 escrowedAccountBalance, VestingEntries.VestingEntry[] memory vestingEntries);
Expand Down
2 changes: 1 addition & 1 deletion contracts/frozen/FuturesMarketBase.sol
Original file line number Diff line number Diff line change
Expand Up @@ -458,7 +458,7 @@ contract FuturesMarketBase is MixinFuturesMarketSettings, IFuturesMarketBaseType
/// Uses the exchanger to get the dynamic fee (SIP-184) for trading from sUSD to baseAsset
/// this assumes dynamic fee is symmetric in direction of trade.
/// @dev this is a pretty expensive action in terms of execution gas as it queries a lot
/// of past rates from oracle. Shoudn't be much of an issue on a rollup though.
/// of past rates from oracle. Shouldn't be much of an issue on a rollup though.
function _dynamicFeeRate() internal view returns (uint feeRate, bool tooVolatile) {
return _exchanger().dynamicFeeRateForExchange(sUSD, baseAsset);
}
Expand Down
2 changes: 1 addition & 1 deletion contracts/interfaces/IRewardEscrowV2.sol
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ interface IRewardEscrowV2 {
VestingEntries.VestingEntry[] calldata vestingEntries
) external;

// Return amount of SNX transfered to SynthetixBridgeToOptimism deposit contract
// Return amount of SNX transferred to SynthetixBridgeToOptimism deposit contract
function burnForMigration(address account, uint256[] calldata entryIDs)
external
returns (uint256 escrowedAccountBalance, VestingEntries.VestingEntry[] memory vestingEntries);
Expand Down
2 changes: 1 addition & 1 deletion examples/browser.html
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<!DOCTYPE html>
<html>
<head>
<title>Example for browser envionments</title>
<title>Example for browser environments</title>
</head>

<body>
Expand Down
2 changes: 1 addition & 1 deletion publish/src/commands/deploy/configure-system-settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -401,7 +401,7 @@ module.exports = async ({
});

const aggregatorWarningFlags = (await getDeployParameter('AGGREGATOR_WARNING_FLAGS'))[network];
// If deploying to OVM avoid ivoking setAggregatorWarningFlags for now.
// If deploying to OVM avoid invoking setAggregatorWarningFlags for now.
if (aggregatorWarningFlags && !useOvm) {
await runStep({
contract: 'SystemSettings',
Expand Down
2 changes: 1 addition & 1 deletion test/etherscan/spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ describe(`Etherscan on ${network}`, () => {
const removeSignaturesAndVariableNames = entry => {
delete entry.signature;
// Some contracts, such as ProxyERC20 were deployed with different function
// input names than currently in the code, so reomve these from the check
// input names than currently in the code, so remove these from the check
// specifically balanceOf(address owner) was changed to balanceOf(address account)
(entry.inputs || []).forEach(input => {
delete input.name;
Expand Down