New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Additional Changes to "Escape quotes in identifiers (#771)" #811

Open

jereze wants to merge 2 commits into kayak:master from jereze:pr/alexandermalyga/771

jereze commented Aug 2, 2024

This pull request builds upon the original PR #771 by @alexandermalyga titled "Escape quotes in identifiers" addressing issue #768.

I confirm that the fix by @alexandermalyga works as intended.
I created additional tests to confirm that escaping quotes in identifiers functions correctly.
I removed FIXME placeholders accordingly after verifying the functionality.

alexandermalyga and others added 2 commits

November 24, 2023 14:06


          Escape quotes in identifiers

2fcb182


          Add more tests to confirm that escaping quotes in identifiers works a…

caecb45

…nd remove FIXME placeholders

jereze requested a review from a team as a code owner

August 2, 2024 14:09

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet