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

Add an architectural overview of bindgen to CONTRIBUTING.md #988

Merged
merged 1 commit into from
Sep 13, 2017

Conversation

fitzgen
Copy link
Member

@fitzgen fitzgen commented Sep 13, 2017

This should help new contributors who are coming to the code base for the first time get up and running.

r? @emilio

This should help new contributors who are coming to the code base for the first
time get up and running.
@emilio
Copy link
Contributor

emilio commented Sep 13, 2017

@bors-servo r+

Thanks a lot!

@bors-servo
Copy link

📌 Commit 4089b3a has been approved by emilio

bors-servo pushed a commit that referenced this pull request Sep 13, 2017
Add an architectural overview of `bindgen` to CONTRIBUTING.md

This should help new contributors who are coming to the code base for the first time get up and running.

r? @emilio
@bors-servo
Copy link

⌛ Testing commit 4089b3a with merge b349cf7...

@fitzgen
Copy link
Member Author

fitzgen commented Sep 13, 2017

Wow! Fast review :)

@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: emilio
Pushing b349cf7 to master...

@bors-servo bors-servo merged commit 4089b3a into rust-lang:master Sep 13, 2017
@fitzgen fitzgen deleted the overview branch September 13, 2017 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants