Skip to content

Remove unused extract_grammar.py #25081

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

Merged
merged 1 commit into from
May 5, 2015
Merged

Remove unused extract_grammar.py #25081

merged 1 commit into from
May 5, 2015

Conversation

carols10cents
Copy link
Member

This script used to be used to extract the grammar sections from the reference, but there is now a separate src/doc/grammar.md that generates grammar.html where the grammar sections that used to be in the reference live, so there is no longer a need to extract the grammar from the reference.

I ❤️ deleting code :) But I totally understand if there's a reason to keep this around that I don't know about :)

This script used to be used to extract the grammar sections from the
reference, but there is now a separate src/doc/grammar.md where the
grammar sections that used to be in the reference live, so there is
no longer a need to extract the grammar from the reference.
@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@steveklabnik
Copy link
Member

Nope, seems good!

@steveklabnik
Copy link
Member

❤️

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented May 3, 2015

📌 Commit 92d49cf has been approved by steveklabnik

@bors
Copy link
Collaborator

bors commented May 4, 2015

⌛ Testing commit 92d49cf with merge 6a3b0ce...

@bors
Copy link
Collaborator

bors commented May 4, 2015

💔 Test failed - auto-mac-32-opt

@alexcrichton
Copy link
Member

@bors: retry

On Mon, May 4, 2015 at 6:16 AM, bors notifications@github.com wrote:

[image: 💔] Test failed - auto-mac-32-opt
http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/4836


Reply to this email directly or view it on GitHub
#25081 (comment).

@carols10cents
Copy link
Member Author

@bors: retry

@carols10cents
Copy link
Member Author

Oh well, was worth a shot ;)

Manishearth added a commit to Manishearth/rust that referenced this pull request May 5, 2015
…r, r=steveklabnik

 This script used to be used to [extract the grammar sections from the reference](rust-lang#8585), but there is [now a separate src/doc/grammar.md](rust-lang#22308) that generates grammar.html where the grammar sections that used to be in the reference live, so there is no longer a need to extract the grammar from the reference.

I ❤️ deleting code :) But I totally understand if there's a reason to keep this around that I don't know about :)
@bors bors merged commit 92d49cf into rust-lang:master May 5, 2015
@carols10cents carols10cents deleted the remove-extract-grammar branch May 5, 2015 13:28
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.

6 participants