Skip to content

feat: non-escape mode (draft for discussion) #862

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

Closed
wants to merge 2 commits into from

Conversation

canalun
Copy link
Contributor

@canalun canalun commented Apr 29, 2023

this is for explanation of the discussion #861

@canalun canalun changed the title chore: non-escape mode (draft for discussion) feat: non-escape mode (draft for discussion) Apr 29, 2023
@canalun canalun closed this May 8, 2023
@canalun
Copy link
Contributor Author

canalun commented May 8, 2023

moved to #870 , but please keep the branch on this PR until #870 is merge because my format tool uses it...!

@canalun canalun deleted the feat/add_non_escape_mode branch May 8, 2023 18:40
@canalun canalun restored the feat/add_non_escape_mode branch May 8, 2023 18:40
@alamb
Copy link
Contributor

alamb commented May 8, 2023

Since the branch is on your fork, there is no need to remove it nor any reason for the sqlparser-rs maintainers to do so either

@canalun
Copy link
Contributor Author

canalun commented May 9, 2023

got it!! thank you for letting me know that!

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.

2 participants