Skip to content
This repository was archived by the owner on Dec 16, 2024. It is now read-only.

[Proposal] Give @kvark reviewer privileges. #551

Merged
merged 1 commit into from
Dec 13, 2016

Conversation

glennw
Copy link
Member

@glennw glennw commented Dec 13, 2016

kvark has make several important contributions to webrender,
both as a reviewer and committer. He also has significant
Rust experience previously as the author of gfx-rs.

Some examples of recent reviews:

servo/webrender#546
servo/webrender#523
servo/webrender#561
servo/webrender#554


This change is Reviewable

kvark has make several important contributions to webrender,
both as a reviewer and committer. He also has significant
Rust experience previously as the author of gfx-rs.

Some examples of recent reviews:

servo/webrender#546
servo/webrender#523
servo/webrender#561
servo/webrender#554
@glennw
Copy link
Member Author

glennw commented Dec 13, 2016

cc @larsbergstrom

@larsbergstrom
Copy link
Contributor

@bors-servo r+

@kvark Thanks for all your awesome work so far, and I'm excited to have you as a new reviewer!

@bors-servo
Copy link
Contributor

📌 Commit ce4e2e3 has been approved by larsbergstrom

@highfive highfive assigned larsbergstrom and unassigned aneeshusa Dec 13, 2016
@bors-servo
Copy link
Contributor

⌛ Testing commit ce4e2e3 with merge 1724ff3...

bors-servo pushed a commit that referenced this pull request Dec 13, 2016
[Proposal] Give @kvark reviewer privileges.

kvark has make several important contributions to webrender,
both as a reviewer and committer. He also has significant
Rust experience previously as the author of gfx-rs.

Some examples of recent reviews:

servo/webrender#546
servo/webrender#523
servo/webrender#561
servo/webrender#554

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/551)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit ce4e2e3 into servo:master Dec 13, 2016
@kvark
Copy link
Member

kvark commented Dec 13, 2016

@larsbergstrom thank you! This ability is going to be helpful to us, at least for small Gecko-related changes.

@larsbergstrom
Copy link
Contributor

@kvark I will deploy it tomorrow morning (US Central time). I try not to make infra changes late at night :-)

@glennw glennw deleted the kvark-reviewer branch December 13, 2016 02:51
@larsbergstrom
Copy link
Contributor

@kvark I lied, because I broke buildbot today for unrelated issues :-/ Tomorrow, really for sure!

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants