Opened 20 months ago

Last modified 18 months ago

#9042 enhancement new

Add treq to Twisted CI

Reported by: Jean-Paul Calderone Owned by: Jean-Paul Calderone
Priority: normal Milestone:
Component: release management Keywords:
Cc: Branch: 9042-ci-for-treq
branch-diff, diff-cov, branch-cov, buildbot
Author:

Description

See ​mailing list discussion.

Change History (5)

comment:1 Changed 20 months ago by Jean-Paul Calderone

Branch: → 9042-ci-for-treq

​https://github.com/twisted/twisted/pull/710

comment:2 Changed 20 months ago by Jean-Paul Calderone

Keywords: review added

comment:3 Changed 18 months ago by Wilfredo Sánchez Vega

Twisted CI is already pretty bloated. Does Twisted really need to be testing downstream projects in its own CI?

This seems like the wrong model here. As are going to add every subproject? Like why not also test klein?

Klein has it's own CI which does test Twisted trunk, though not on every Twisted commit on trunk. So that's not as complete, but this feels like overkill.

comment:4 Changed 18 months ago by hawkowl

Keywords: review removed
Owner: set to Jean-Paul Calderone

Thanks, JP. See the review on the PR.

comment:5 Changed 18 months ago by hawkowl

(which, to deduplicate here, was 'please make the builds not fail until we have a patch, then merge' :) )

Note: See TracTickets for help on using tickets.