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

deployer_whitellist.go: Grammar #1654

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
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
16 changes: 8 additions & 8 deletions builtin/plugins/deployer_whitelist/deployer_whitelist.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ type (
)

const (
// AllowEVMDeployFlag indicates that a deployer is permitted to deploy EVM contract.
// AllowEVMDeployFlag indicates that a deployer is allowed to deploy EVM contracts.
AllowEVMDeployFlag = dwtypes.Flags_EVM
// AllowGoDeployFlag indicates that a deployer is permitted to deploy GO contract.
// AllowGoDeployFlag indicates that a deployer is allowed to deploy GO contracts.
AllowGoDeployFlag = dwtypes.Flags_GO
// AllowMigrationFlag indicates that a deployer is permitted to migrate GO contract.
// AllowMigrationFlag indicates that a deployer is allowed to migrate GO contracts.
AllowMigrationFlag = dwtypes.Flags_MIGRATION
)

Expand All @@ -42,11 +42,11 @@ var (
// ErrInvalidRequest is a generic error that's returned when something is wrong with the
// request message, e.g. missing or invalid fields.
ErrInvalidRequest = errors.New("[DeployerWhitelist] invalid request")
// ErrOwnerNotSpecified returned if init request does not have owner address
// ErrOwnerNotSpecified is returned if the init request does not have the owner address
ErrOwnerNotSpecified = errors.New("[DeployerWhitelist] owner not specified")
// ErrFeatureFound returned if an owner try to set an existing feature
// ErrFeatureFound is returned if an owner tries to set an existing feature
ErrDeployerAlreadyExists = errors.New("[DeployerWhitelist] deployer already exists")
// ErrDeployerDoesNotExist returned if an owner try to to remove a deployer that does not exist
// ErrDeployerDoesNotExist is returned if an owner tries to remove a deployer that does not exist
ErrDeployerDoesNotExist = errors.New("[DeployerWhitelist] deployer does not exist")
)

Expand Down Expand Up @@ -103,7 +103,7 @@ func (dw *DeployerWhitelist) Init(ctx contract.Context, req *InitRequest) error
func (dw *DeployerWhitelist) AddUserDeployer(ctx contract.Context, req *AddUserDeployerRequest) error {
userWhitelistContract, err := ctx.Resolve("user-deployer-whitelist")
if err != nil {
return errors.Wrap(err, "unable to resolve user_deployer_whitelist contract")
return errors.Wrap(err, "unable to resolve the user_deployer_whitelist contract")
}

if ctx.Message().Sender.Compare(userWhitelistContract) != 0 {
Expand Down Expand Up @@ -136,7 +136,7 @@ func (dw *DeployerWhitelist) RemoveUserDeployer(ctx contract.Context, req *Remov
//check if authorized
userWhitelistContract, err := ctx.Resolve("user-deployer-whitelist")
if err != nil {
return errors.Wrap(err, "unable to resolve user_deployer_whitelist contract")
return errors.Wrap(err, "unable to resolve the user_deployer_whitelist contract")
}

if ctx.Message().Sender.Compare(userWhitelistContract) != 0 {
Expand Down