Opened 2 years ago

Closed 2 years 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:
Severity: Normal Keywords: tor-ci, 034-backport
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Appveyor currently reports:

- cmd: ... success
- cmd: ... failure

which is really noisy.

Travis currently reports:

    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 2 years ago by teor

Status: newneeds_information

Please see my branch ticket27275 on

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

comment:2 Changed 2 years 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 2 years ago by catalyst

I agree with making appveyor quieter.

comment:4 Changed 2 years ago by nickm

How far should I backport this?

comment:5 Changed 2 years ago by teor

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

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

See my branch ticket27275-034, ci here:

comment:6 Changed 2 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Merged to 0.3.4 and forward!

Note: See TracTickets for help on using tickets.