ms705 / nom-sql

Rust SQL parser written using nom
MIT License
232 stars 41 forks source link

Add implementation for delete and update statement #9

Closed ivanceras closed 7 years ago

ivanceras commented 7 years ago

This change is Reviewable

ivanceras commented 7 years ago

For issue https://github.com/ms705/nom-sql/issues/2

ivanceras commented 7 years ago

I think making a PR for the additional sql statements then the impl-display later would be much more concise, since there would be impl-display for the update and delete statement. Please review and hopefully merge this PR as soon as you guys have time for it, so I can follow-up the impl-display as my next PR which depends on merge.

ms705 commented 7 years ago

:+1: LGTM!

Thanks for splitting the prior PR up -- agreed it's better to have these separated out.