Skip to content

Conversation

@jfelchner
Copy link
Contributor

When working with settings verification, it's
important to remember to let the Approver know that settings have
changed and that the current file signature will no longer match.

You can enable it by creating an .overcommit.yml file and adding:

ChamberVerification:
  enabled: true

Whenever you commit any of your settings files, the hook will warn the
user that the signature no longer matches and that the Approver needs to
be notified.

If you'd like to specify the name and email address of the Approver, you
can add that to the configuration:

ChamberVerification:
  enabled:        true
  approver_name:  "Jeff Felchner"
  approver_email: "jeff@example.com"

@sds sds mentioned this pull request Jan 19, 2018
@jfelchner
Copy link
Contributor Author

@sds Rebased. Thank you. :)

@sds
Copy link
Owner

sds commented Jan 19, 2018

Hey @jfelchner, looks like you accidentally removed some configuration from another (unrelated) PR. Can you rebase again? Thanks!

When working with [settings verification](Verifying-Settings), it's
important to remember to let the Approver know that settings have
changed and that the current file signature will no longer match.

You can enable it by creating an `.overcommit.yml` file and adding:

```yaml
ChamberVerification:
  enabled: true
```

Whenever you commit any of your settings files, the hook will warn the
user that the signature no longer matches and that the Approver needs to
be notified.

If you'd like to specify the name and email address of the Approver, you
can add that to the configuration:

```yaml
ChamberVerification:
  enabled:        true
  approver_name:  "Jeff Felchner"
  approver_email: "jeff@example.com"
```
@jfelchner
Copy link
Contributor Author

Ahhhhhhh!!!! Sorry man. Should be good now.

@sds sds merged commit 8ee8324 into sds:master Jan 21, 2018
@jfelchner
Copy link
Contributor Author

@sds any chance we can get a release out soon? 😁

@sds
Copy link
Owner

sds commented Jan 30, 2018

0.42.0 was just released!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants