Mention what build.rs is for and why it's commented out in Cargo.toml #308

Activity

1. bajr commented on Aug 26, 2016

   

   bajr

   on Aug 26, 2016

   Contributor

   More actions

   Would it be worth removing these and just documenting how to build the parser manually in the contributing doc? I feel like it might be the more long-term solution for future edits to the parser, rather than having contributors edit the Cargo.toml each time they want to rebuild the grammar.
2. selenesal commented on Aug 26, 2016

   

   selenesal

   on Aug 26, 2016

   ContributorAuthor

   More actions

   That could certainly be an option, but manually building it is a little tedious, especially when it can all be done in one step with Cargo. When you're making multiple changes to the grammar, I think it's more convenient.
3. arc3x commented on Aug 26, 2016

   

   arc3x

   on Aug 26, 2016

   Contributor

   More actions

   I think it should be left in with a note of what it is for (in code & documentation)