PrePAN

Sign in to PrePAN

CI Get details about the current CI environment

Author
skaji@github
Date
URL
https://github.com/skaji/CI
Status
In Review
Good

Synopsis

use Test::More;
use CI qw(is_ci);

plan skip_all => 'only in CI' if !is_ci;

...

done_testing;

Description

This module provides details about the current CI environment. This is a perl port of https://github.com/watson/ci-info.

Questions:

  • Do you think the name "CI" is appropriate?
  • Do you think this module is useful?
  • Any feedback is welcome.

Thanks.

Comments

I think it would be better named with a qualifier after `CI` (e.g. `CI::Generic`) that way others can supply similar packages for other CI setups that yours doesn't support yet.

Do I think it's useful? If you only use one CI setup then it's probably just as easy to code directly against that. However, for anyone who has to deal with multiple CI setups, or those who are thinking they might change their CI setup then providing a standardized interface that decouples your code from your CI setup would certainly be beneficial.
anonymouse
Anonymous
The CI acronym can stand for other things, so it's probably not a good idea to use it as a top-level namespace.
> jlcooper@github

Thanks!

> I think it would be better named with a qualifier after `CI` (e.g. `CI::Generic`) that way others can supply similar packages for other CI setups that yours doesn't support yet.

Good point.

---

> Anonymous

Thanks!

> it's probably not a good idea to use it as a top-level namespace.

I see.

Please sign up to post a review.