-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Conversation
This patch add CI information on the user_agent in order for us to track if a given command was run by a CI, as discussed in rubygems#3233.
|
||
describe "include CI information" do | ||
it "from one CI" do | ||
ENV["JENKINS_URL"] = "foo" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this ENV should be cleaned
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done! Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will be better to keep and reassign old value (instead of forcing nil).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of course! Silly me!
Can I have this job on travis run again? I just tried locally and it worked with ruby 2.2.2. Looking at the travis log, it seems like a transient error. It have messages like this:
|
restarted the build, I think it was failing because RubyGems master was broken |
@indirect Thank you! |
Thanks! |
This patch add CI information on the user_agent in order for us to track
if a given command was run by a CI, as discussed in #3233.