-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #24 from appsignal/add-name-argument
Add name argument
- Loading branch information
Showing
4 changed files
with
337 additions
and
109 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
--- | ||
bump: minor | ||
type: change | ||
--- | ||
|
||
Add a required positional argument for the name. This name is used as the identifier for cron and heartbeat check-ins, the group for logs, and the action name for errors. | ||
|
||
This avoids repetition of command-line parameters that represent the name: | ||
|
||
```sh | ||
# Before: | ||
appsignal-wrap \ | ||
--cron backup \ | ||
--error backup \ | ||
--log backup \ | ||
-- ./backup.sh | ||
|
||
# After: | ||
appsignal-wrap backup \ | ||
--cron \ | ||
-- ./backup.sh | ||
``` | ||
|
||
It is still possible to override the name for a specific purpose by using the `--log GROUP` and `--error ACTION` arguments, or by passing an identifier to either `--cron` or `--heartbeat`: | ||
|
||
```sh | ||
appsignal-wrap mysql \ | ||
--heartbeat db | ||
-- mysqld | ||
``` | ||
|
||
Additionally, error sending is now enabled by default (use `--no-error` to disable it) and using both cron and heartbeat check-ins in the same invocation is no longer allowed. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.