Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix deprecated plugin APIs #6156

Merged
merged 1 commit into from May 21, 2015
Merged

Fix deprecated plugin APIs #6156

merged 1 commit into from May 21, 2015

Conversation

@Manishearth
Copy link
Member

Manishearth commented May 21, 2015

r? @larsbergstrom

(The diff is borked here, but I only just added if lets and wrapped/unwrapped in Annotatables)

Review on Reviewable

@hoppipolla-critic-bot
Copy link

hoppipolla-critic-bot commented May 21, 2015

Critic review: https://critic.hoppipolla.co.uk/r/5072

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@larsbergstrom
Copy link
Contributor

larsbergstrom commented May 21, 2015

Review status: :shipit: all files reviewed, all discussions resolved, all commit checks successful.
Reviewed files:

  • components/plugins/jstraceable.rs @ r1
  • components/plugins/lib.rs @ r1
  • components/plugins/reflector.rs @ r1

Comments from the review on Reviewable.io

@larsbergstrom larsbergstrom self-assigned this May 21, 2015
@larsbergstrom
Copy link
Contributor

larsbergstrom commented May 21, 2015

@bors-servo
Copy link
Contributor

bors-servo commented May 21, 2015

📌 Commit 8394b82 has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented May 21, 2015

Testing commit 8394b82 with merge ef7fa99...

bors-servo pushed a commit that referenced this pull request May 21, 2015
r? @larsbergstrom


(The diff is borked here, but I only just added `if let`s and wrapped/unwrapped in `Annotatable`s)

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6156)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 21, 2015

☀️ Test successful - android, gonk, linux1, linux2, mac1, mac2

@bors-servo bors-servo merged commit 8394b82 into servo:master May 21, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Manishearth Manishearth deleted the Manishearth:warn_fix branch Sep 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.