search

barosl / homu

A bot that integrates with GitHub and your favorite continuous integration service
http://homu.io/
MIT License
661 stars 108 forks source link

Upstream changes from Servo #133

Closed Manishearth closed 8 years ago

Manishearth commented 8 years ago

This contains changes which add flake8 linting to the CI, and a fix for #100.

The multiple build thing seems to work fine for us with appveyor.

Let me know if I should exclude something! I'd like servo/homu and homu/homu to be near-identical, with servo being a bit more liberal in accepting fixes which get eventually upstreamed to homu after style decisions and testing is done.

r? @barosl

Manishearth commented 8 years ago

cc @frewsxcv @larsbergstrom

sigmavirus24 commented 8 years ago

This is a giant change in a single pull request. Also, nitpicking: Isn't barosl/homu upstream and servo a downstream fork? ;)

Manishearth commented 8 years ago

@sigmavirus24 It isn't, there are only two changes in this PR (the rest are merge pull requests).

Yes, barosl/homu is upstream, hence "upstreaming". I'm using it as a verb.

Manishearth commented 8 years ago

Superseded by #133

frewsxcv commented 8 years ago

Superseded by https://github.com/barosl/homu/issues/134