opencypher / openCypher

Specification of the Cypher property graph query language
http://www.opencypher.org
Apache License 2.0
841 stars 149 forks source link

Commit the .ebnf output into the repo #540

Open jacobfriedman opened 1 year ago

jacobfriedman commented 1 year ago

Please include the generated opencypher EBNF file in the repo

Otherwise, for those relying on the .ebnf file, for every update in the grammar those people would have to regenerate the file and check the diff.

hvub commented 1 year ago

Thanks for using with the openCypher artifacts.

It is a general practice of the openCypher repo to not include built artifacts. Exceptions to that rules require very good reason.

The reason you are providing is not convincing. I do not see why you have to check the diff of a rebuilt EBNF. Why not just using the regenerate EBNF? Conversely, if the actual EBNF diff is an essential input to your usage of the grammar and if we would include the EBNF into the repo, then you would still will have the obtain the diff of the EBNF. So, I can not see any benefit regarding the diff.

If it is the regeneration of the EBNF, i.e. running the build, that bothers you, I am afraid I can not help you either. The repo contains the source code and not the generated artifacts. The generated artifacts are released in milestones. This gives you two options to chose from: