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 the grammar from the reference #24729

Merged
merged 3 commits into from Apr 25, 2015
Merged

Conversation

@mdinger
Copy link
Contributor

mdinger commented Apr 23, 2015

From #24723 (comment) :

Yes, I've been meaning to just remove it in favor of GRAMMAR.md

  • steveklabnik
@rust-highfive
Copy link
Collaborator

rust-highfive commented Apr 23, 2015

r? @pcwalton

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

@mdinger
Copy link
Contributor Author

mdinger commented Apr 23, 2015

@mdinger mdinger force-pushed the mdinger:reference_grammar branch from 3632de3 to 02bd862 Apr 23, 2015
@mdinger
Copy link
Contributor Author

mdinger commented Apr 23, 2015

Updated with a few more removals. Searching for ebnf didn't find anything else but text may talk about the grammar still. You won't discover that without reading though.

@bors
Copy link
Contributor

bors commented Apr 24, 2015

The latest upstream changes (presumably #24759) made this pull request unmergeable. Please resolve the merge conflicts.

@mdinger mdinger force-pushed the mdinger:reference_grammar branch from 02bd862 to 54485dc Apr 24, 2015
@mdinger
Copy link
Contributor Author

mdinger commented Apr 24, 2015

Rebased. Basically deletes the ebnf from the reference and moves some of the recent reference changes to the grammar. Until this is done, the reference is probably have a tendency to be more up to date than the grammar because that's what people look at.

The reference was definitely different at a few places suggesting more recent work.

@mdinger
Copy link
Contributor Author

mdinger commented Apr 24, 2015

Let me know when/if you want this. I'm gonna hold off on rebasing until then because I'll just need to keep doing it.

@steveklabnik
Copy link
Member

steveklabnik commented Apr 24, 2015

I do in the general case, yeah, but with the reference editing, people are changing it.

@mdinger
Copy link
Contributor Author

mdinger commented Apr 24, 2015

I can roll up the reference changes into one I think. That might work.

@mdinger mdinger force-pushed the mdinger:reference_grammar branch from bc53811 to 8f2c399 Apr 24, 2015
@mdinger
Copy link
Contributor Author

mdinger commented Apr 24, 2015

@steveklabnik This rolls all the reference changes into one PR

@steveklabnik
Copy link
Member

steveklabnik commented Apr 24, 2015

I'd rather let bors handle all those normally rather than doing some kind of manual rollup

@mdinger mdinger force-pushed the mdinger:reference_grammar branch from 8f2c399 to 7089c5e Apr 24, 2015
@mdinger
Copy link
Contributor Author

mdinger commented Apr 24, 2015

Okay. Reverted. I only did it because I was afraid of merge conflicts. I don't know how bors handles them.

@steveklabnik
Copy link
Member

steveklabnik commented Apr 24, 2015

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Apr 24, 2015

📌 Commit 7089c5e has been approved by steveklabnik

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 24, 2015
…labnik

From rust-lang#24723 (comment) :

> Yes, I've been meaning to just remove it in favor of GRAMMAR.md
> - steveklabnik
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 24, 2015
…labnik

From rust-lang#24723 (comment) :

> Yes, I've been meaning to just remove it in favor of GRAMMAR.md
> - steveklabnik
bors added a commit that referenced this pull request Apr 25, 2015
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 25, 2015
…labnik

From rust-lang#24723 (comment) :

> Yes, I've been meaning to just remove it in favor of GRAMMAR.md
> - steveklabnik
bors added a commit that referenced this pull request Apr 25, 2015
- Successful merges: #24662, #24722, #24725, #24729, #24736, #24749, #24751, #24766, #24769, #24772, #24775, #24790
- Failed merges: #24760
@bors bors merged commit 7089c5e into rust-lang:master Apr 25, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mdinger mdinger deleted the mdinger:reference_grammar branch Apr 25, 2015
@mdinger mdinger mentioned this pull request Apr 25, 2015
59 of 150 tasks complete
bors added a commit that referenced this pull request Apr 30, 2015
I'm interested in helping out with #16676 but more in the grammar than the reference-- here's my first chunk, more to come!! 🎉 

I did pull a bit *out* of the reference, though, that was more relevant to the grammar but wasn't moved over as part of #24729.

I'm looking at, e.g. https://github.com/rust-lang/rust/blob/master/src/libsyntax/ast.rs, as the source of truth, please let me know if I should be checking against something else instead/in addition.

r? @steveklabnik
bors added a commit that referenced this pull request Apr 30, 2015
I'm interested in helping out with #16676 but more in the grammar than the reference-- here's my first chunk, more to come!! 🎉 

I did pull a bit *out* of the reference, though, that was more relevant to the grammar but wasn't moved over as part of #24729.

I'm looking at, e.g. https://github.com/rust-lang/rust/blob/master/src/libsyntax/ast.rs, as the source of truth, please let me know if I should be checking against something else instead/in addition.

r? @steveklabnik
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.