Replace duration with elapsed_time in Slack Notifications

[Jacobkg]

This is probably controversial, but I would argue better in almost all cases

See, e.g travis-ci/travis-ci#5811

@BanzaiMan

[joshk]

Ohhhhhh I'm torn on this. It makes sense and could be a good change, but
then maybe we should do it everywhere?

Also, we have found a small bug with elapsed time and should maybe wait
until that is fixed.

On Fri, 8 Apr 2016 at 5:12 PM, Josh Kalderimis [email protected]
wrote:

I
On Fri, 8 Apr 2016 at 5:10 PM, Jacob Green [email protected]
wrote:

This is probably controversial, but I would argue this is better.

See, e.g travis-ci/travis-ci#5811
travis-ci/travis-ci#5811

@BanzaiMan https://github.com/BanzaiMan

You can view, comment on, or merge this pull request online at:

#57
Commit Summary

• Elapsed time is arguably more useful
• Merge remote-tracking branch 'travis-ci/master'

File Changes

• M lib/travis/addons/slack/task.rb
  https://github.com/travis-ci/travis-tasks/pull/57/files#diff-0 (4)

Patch Links:

• https://github.com/travis-ci/travis-tasks/pull/57.patch
• https://github.com/travis-ci/travis-tasks/pull/57.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#57

[Jacobkg]

@joshk What's the bug?

[Jacobkg]

@joshk @BanzaiMan Any followup on this?