Opened 8 weeks ago

Closed 8 weeks ago

#27275 closed defect (fixed)

Stop reporting appveyor on_success, because it's noisy

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version: Tor: 0.3.4.1-alpha
Severity: Normal Keywords: tor-ci, 034-backport
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Appveyor currently reports:

on_success:
- cmd: ... success
on_failure:
- cmd: ... failure

which is really noisy.

Travis currently reports:

  irc:
    ...
    on_success: change
    on_failure: change

which seems ok.

We should make Appveyor notifications more like Travis.

Child Tickets

Change History (6)

comment:1 Changed 8 weeks ago by teor

Status: newneeds_information

Please see my branch ticket27275 on https://github.com/teor2345/tor.git

I'm going to make sure that people actually want this change before I get it reviewed.

comment:2 Changed 8 weeks ago by teor

Status: needs_informationneeds_review

nickm says yes, so I'll let this ticket go through the review process while others respond.

comment:3 Changed 8 weeks ago by catalyst

I agree with making appveyor quieter.

comment:4 Changed 8 weeks ago by nickm

How far should I backport this?

comment:5 Changed 8 weeks ago by teor

Keywords: 034-backport added
Milestone: Tor: unspecifiedTor: 0.3.5.x-final
Version: Tor: 0.3.4.1-alpha

.appveyor.yml was introduced in 0.3.4, so we can just cherry-pick backwards.

See my branch ticket27275-034, ci here:
https://ci.appveyor.com/project/teor2345/tor/build/1.0.87

comment:6 Changed 8 weeks ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Merged to 0.3.4 and forward!

Note: See TracTickets for help on using tickets.