• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

oconnor663 / duct / 109
100%

Build:
DEFAULT BRANCH: master
Ran 04 Mar 2016 05:45AM UTC
Jobs 8
Files 1
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
109

push

travis-ci

oconnor663
get rid of STRING and BYTES in favor of PIPE

I think it will be fairly common for people to want to `read()` stdout
as bytes. Requiring callers to `from duct import BYTES` feels
unnecessarily awkward. Setting `decode=False` is much easier, and it
avoids the confusing state where one output is decoded and the other
isn't.

There's a chance that `PIPE` itself will feel too close to `pipe()` for
comfort, given that it means something very different. I might want to
bikeshed that some more in the future. (Maybe simpler flags like
`capture_stdout` and `capture_stderr`? Though it would be a little weird
to have those and also `stdout`/stderr`.) But in any case `PIPE` is the
Python convention, and it's a reasonable first improvement.

While we're at it, rename `trim` to `sh_trim`. That's more suggestive of
what it's actually doing, especially with an eye towards languages (like
Rust) that use `trim` to mean what `strip` means in Python.

Closes https://github.com/oconnor663/duct/issues/32.

338 of 342 relevant lines covered (98.83%)

7.83 hits per line

Jobs
ID Job ID Ran Files Coverage
1 109.1 04 Mar 2016 05:45AM UTC 0
98.25
Travis Job 109.1
2 109.2 04 Mar 2016 05:45AM UTC 0
98.25
Travis Job 109.2
3 109.3 04 Mar 2016 05:45AM UTC 0
97.66
Travis Job 109.3
4 109.4 04 Mar 2016 05:46AM UTC 0
97.37
Travis Job 109.4
5 109.5 04 Mar 2016 05:46AM UTC 0
97.37
Travis Job 109.5
6 109.6 04 Mar 2016 05:47AM UTC 0
98.25
Travis Job 109.6
7 109.7 04 Mar 2016 05:46AM UTC 0
98.25
Travis Job 109.7
8 109.8 04 Mar 2016 05:46AM UTC 0
97.37
Travis Job 109.8
Source Files on build 109
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #109
  • b0299652 on github
  • Prev Build on master (#108)
  • Next Build on master (#110)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc