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

Remove feature gated examples from the reference #24577

Merged
merged 1 commit into from
Apr 21, 2015

Conversation

steveklabnik
Copy link
Member

Fixes #24573

@rust-highfive
Copy link
Collaborator

r? @pcwalton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

I can see where we may want to remove as many feature-gated examples as possible, but some of these seem like they may still be useful? For example the one showing off slice patterns is probably good to keep.

@pnkfelix
Copy link
Member

maybe we should move them to an isolated section of the documentation? That way the examples are preserved, without having them in the middle of the text that users on the beta or release channels are reading?

@steveklabnik
Copy link
Member Author

maybe we should move them to an isolated section of the documentation?

Yes, so the reason I just killed these examples is that we have a specific unstable section of the book for describing unstable features. And I'm not sure it's worth time finding alternate examples of things when the reference needs a general cleaning up anyway, which is where I'd go over things like 'does this deserve an example'

@alexcrichton
Copy link
Member

Ok, but some of these I believe can be updated to not use feature gates, so perhaps they could be tweaked to stay wherever possible?

@steveklabnik steveklabnik force-pushed the gh24573 branch 2 times, most recently from 1966ca3 to 0f20c9d Compare April 20, 2015 17:33
The slice patterns example was nice, so let's put it with the other
slice pattern feature gate documentation.

Fixes rust-lang#24573
@steveklabnik
Copy link
Member Author

@alexcrichton seems legit. I've fixed up the examples, and moved the slice patterns one out. Only one or two got totally axed, because they were about box patterns, primarily.

@alexcrichton
Copy link
Member

@bors: r+ 0f20c9d rollup

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 20, 2015
bors added a commit that referenced this pull request Apr 20, 2015
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 21, 2015
bors added a commit that referenced this pull request Apr 21, 2015
@bors bors merged commit 0f20c9d into rust-lang:master Apr 21, 2015
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.

Examples in beta docs using unstable features
6 participants