Skip to content

Add first test #10

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.

Sign up for GitHub

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

Jump to bottom
Merged
fabricedesre merged 2 commits into from
Feb 3, 2016
Merged

Add first test #10

fabricedesre merged 2 commits into from
Feb 3, 2016

Conversation

@JohanLorenzo
Copy link
Contributor

@JohanLorenzo JohanLorenzo commented Feb 3, 2016

I mess up the original PR #1. Here's a correct one.

JohanLorenzo
JohanLorenzo reviewed Feb 3, 2016
View reviewed changes
src/context.rs
Copy link
Contributor Author

@JohanLorenzo JohanLorenzo Feb 3, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fabricedesre to_owned() + inline done.

fabricedesre added a commit that referenced this pull request Feb 3, 2016
@fabricedesre
Merge pull request #10 from JohanLorenzo/add-first-test
75484bf 
Add first test
@fabricedesre fabricedesre merged commit 75484bf into fxbox:master Feb 3, 2016
@JohanLorenzo JohanLorenzo deleted the add-first-test branch March 4, 2016 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JohanLorenzo @fabricedesre