Closed mstange closed 3 years ago
@mstange Is that https://github.com/servo/core-foundation-rs/ and https://github.com/servo/gleam/, or are there any others? It would be cleaner to add you to the reviewers list for the specific crates.
Yes, those two, and core-graphics-rs + core-text-rs.
Ok, if you could:
extra_reviewers
list with your name in https://github.com/servo/saltfs/blob/da8e2c6d9f020e33d0fa783a70f65f60ffef7b1e/homu/files/cfg.toml#L41 and https://github.com/servo/saltfs/blob/da8e2c6d9f020e33d0fa783a70f65f60ffef7b1e/homu/files/cfg.toml#L60
then we should be ok to merge.(homu only cares about github repositories, and all the macOS crates live in a workspace in the core-foundation-rs repo now)
Ah, I see. I'll make that change.
I've updated the PR.
@bors-servo r+
:pushpin: Commit 3550f84 has been approved by jdm
:hourglass: Testing commit 3550f84a71a95820fa29a937ba47abe4d07817b3 with merge ee857f6fe028ac083f093399d64afa1543450a79...
:broken_heart: Test failed - checks-travis
@bors-servo r+
:bulb: This pull request was already approved, no need to approve it again.
retry
to trigger a build again.:pushpin: Commit 3550f84 has been approved by jdm
:hourglass: Testing commit 3550f84a71a95820fa29a937ba47abe4d07817b3 with merge 77e01b1578ee7d152d94e89656feee10ad63f31a...
:sunny: Test successful - checks-travis Approved by: jdm Pushing 77e01b1578ee7d152d94e89656feee10ad63f31a to master...
I would like to be able to review changes to the various macOS and OpenGL crates that WebRender depends on.
r? @jdm