Skip to content

Conversation

@gabrielschulhof
Copy link

No description provided.

@gabrielschulhof
Copy link
Author

@mhdawson @gireeshpunathil @benjamingr I have moved the guide to this repo so it might appear on the Web site. Could you PTAL?

Copy link
Member

@benjamingr benjamingr left a comment

Choose a reason for hiding this comment

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

Hey, I didn't get a chance to respond to the feedback initially but this is much better and I still really like having this guide.

Nice work! <3

@vsemozhetbyt vsemozhetbyt added the content Issues/pr concerning content label Oct 7, 2018
@gabrielschulhof
Copy link
Author

@benjamingr thanks!

@gabrielschulhof gabrielschulhof force-pushed the add-abi-stability-guide branch from 9467531 to e98c3a5 Compare October 7, 2018 15:56
@gabrielschulhof
Copy link
Author

Whoops! Forgot to add to index.md.

Copy link
Member

Choose a reason for hiding this comment

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

I guess these links don't work.

@gabrielschulhof gabrielschulhof force-pushed the add-abi-stability-guide branch from e98c3a5 to af53983 Compare October 8, 2018 02:54
@gabrielschulhof
Copy link
Author

@lpinca I have addressed your comments.

gabrielschulhof pushed a commit that referenced this pull request Oct 9, 2018
PR-URL: #1828
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Milos Jevtovic <[email protected]>
@gabrielschulhof
Copy link
Author

Landed in 3214a8d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content Issues/pr concerning content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants