Skip to content
This repository was archived by the owner on Feb 22, 2018. It is now read-only.

Update langtour > interfaces #5

Closed
wants to merge 1 commit into from
Closed

Update langtour > interfaces #5

wants to merge 1 commit into from

Conversation

butlermatt
Copy link
Contributor

Reflect removal of interface keyword, and removal of 'default' classes for interfaces.

Interfaces are types that define how you can interact with an object.
An interface can specify methods,
An interface may specify methods,
Copy link
Contributor

Choose a reason for hiding this comment

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

Our general style is to use can.

@sethladd
Copy link
Contributor

Thanks Matt! The timing of this is challenging, we're going to publish a significant rewrite of this guide, which will make your changes redundant. We really like your contributions. Let's meet over a Hangout to coordinate, there's lots of other places where you can help. :)

@butlermatt
Copy link
Contributor Author

Thanks Seth. I wasn't sure of this change myself to be honest due to the extent of the changes rather than just a coupe of typo fixes. But thought I'd give it a hook anyways. Truthfully as you said this section deserves a full rewrite rather than just an adaptation I attempted.

I'd be happy to me up on hangout sometime to discuss other ways I may be able to help out.

@butlermatt
Copy link
Contributor Author

Since the re-write for this guide is already in the pipe-line would you like to (or like me to) close this pull request? Alternatively I can make some of the changes you suggested (for which I appreciate as well, as it provides insight for future patches as well).

Thanks,
Matt

@sethladd
Copy link
Contributor

I hate to say it, but let's close this request. We'll sync to get you involved in other areas of the site. Thanks again for the patch!

@sethladd sethladd closed this Sep 12, 2012
@butlermatt
Copy link
Contributor Author

I'm perfectly ok with closing it. Makes it easier for me to send a new pull request for something different (and smaller ;)
As I mentioned I was aware of the work that really should have had anyways, but thanks again for the comments on it. They help significantly.
Matt

chrisbu added a commit to chrisbu/dartlang.org that referenced this pull request Nov 10, 2012
sethladd added a commit that referenced this pull request Nov 12, 2012
Actioning comment #5 on dartbug #6470
alanjoyce pushed a commit to alanjoyce/dartlang.org that referenced this pull request Nov 7, 2013
Minor fixes (homepage content, dart-tips thumbnails, and /docs/ permalinks)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants