Closed Lowercases closed 5 years ago
Add the timezone and the author's name to the releases output.
The TZ change is exclusively a client change; for the author to be visible both the client and server must be upgraded (no error if any isn't).
Example output
$ emp-local releases -a acme-inc v2 Russell B… Feb 25 22:40 (-03) setting up the repo structure v3 Fake McFa… Feb 26 00:36 (-03) Deployed remind101/acme-inc:latest v4 Fake McFa… Feb 26 00:47 (-03) Deployed remind101/acme-inc:latest v5 Fake McFa… Feb 26 01:24 (-03) Deployed remind101/acme-inc:latest v6 Fake McFa… Feb 26 01:29 (-03) Deployed remind101/acme-inc:latest v7 Fake McFa… Feb 26 01:49 (-03) Deployed remind101/acme-inc:latest v8 Fake McFa… Mar 13 17:02 (-03) Changed environment variables on acme-inc (TACO) $ TZ=EST emp-local releases -a acme-inc v2 Russell B… Feb 25 20:40 (EST) setting up the repo structure v3 Fake McFa… Feb 25 22:36 (EST) Deployed remind101/acme-inc:latest v4 Fake McFa… Feb 25 22:47 (EST) Deployed remind101/acme-inc:latest v5 Fake McFa… Feb 25 23:24 (EST) Deployed remind101/acme-inc:latest v6 Fake McFa… Feb 25 23:29 (EST) Deployed remind101/acme-inc:latest v7 Fake McFa… Feb 25 23:49 (EST) Deployed remind101/acme-inc:latest v8 Fake McFa… Mar 13 15:02 (EST) Changed environment variables on acme-inc (TACO
The reason for using author instead of committer for user id and email is that the merge might be done by other person; in our case a bot, getting:
$ emp-staging releases -a r101-etl v325 GitHub Feb 19 19:19 (-03) Migrate batch 8 to empire v2 (#1999) v326 GitHub App Feb 19 21:26 (-03) Deployed remind101/r101-etl:3b8b609f5b7b5fcd322f30905b9d6d1e9d1df4b8
instead of the expected
$ emp-staging releases -a r101-etl v325 Russell B… Feb 19 19:19 (-03) Migrate batch 8 to empire v2 (#1999) v326 dalyons Feb 19 21:26 (-03) Deployed remind101/r101-etl:3b8b609f5b7b5fcd322f30905b9d6d1e9d1df4b8
Add the timezone and the author's name to the releases output.
The TZ change is exclusively a client change; for the author to be visible both the client and server must be upgraded (no error if any isn't).
Example output
The reason for using author instead of committer for user id and email is that the merge might be done by other person; in our case a bot, getting:
instead of the expected