Skip to content

chore: finish lexer implementation #1409

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

Merged
merged 1 commit into from
Mar 10, 2025
Merged

chore: finish lexer implementation #1409

merged 1 commit into from
Mar 10, 2025

Conversation

benfdking
Copy link
Collaborator

No description provided.

@benfdking benfdking force-pushed the chore/testing_fix_lexer branch from fad7180 to dc1c741 Compare March 10, 2025 18:51
Copy link

Benchmark for 00b409a

Click to view benchmark
Test Base PR %
DepthMap::from_parent 38.4±0.39µs 38.2±0.46µs -0.52%
fix_complex_query 9.5±0.10ms 9.5±0.65ms 0.00%
fix_superlong 103.8±4.93ms 96.2±4.67ms -7.32%
parse_complex_query 2.9±0.05µs 2.9±0.05µs 0.00%
parse_expression_recursion 5.1±0.10µs 5.1±0.06µs 0.00%
parse_simple_query 807.5±18.41ns 812.1±11.73ns +0.57%

@benfdking benfdking merged commit 9e2af43 into main Mar 10, 2025
28 checks passed
@benfdking benfdking deleted the chore/testing_fix_lexer branch March 10, 2025 19:09
Copy link

Benchmark for 53067a6

Click to view benchmark
Test Base PR %
DepthMap::from_parent 38.5±0.28µs 38.6±0.24µs +0.26%
fix_complex_query 9.4±0.13ms 9.5±0.50ms +1.06%
fix_superlong 94.0±4.97ms 93.1±3.45ms -0.96%
parse_complex_query 2.9±0.04µs 2.9±0.03µs 0.00%
parse_expression_recursion 5.1±0.26µs 5.1±0.05µs 0.00%
parse_simple_query 883.2±9.25ns 807.0±13.60ns -8.63%

tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Mar 11, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [quarylabs/sqruff](https://github.com/quarylabs/sqruff) | patch | `v0.25.14` -> `v0.25.15` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>quarylabs/sqruff (quarylabs/sqruff)</summary>

### [`v0.25.15`](https://github.com/quarylabs/sqruff/releases/tag/v0.25.15)

[Compare Source](quarylabs/sqruff@v0.25.14...v0.25.15)

#### What's Changed

-   chore: bump esbuild from 0.25.0 to 0.25.1 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1399
-   chore: bump tempfile from 3.17.1 to 3.18.0 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1406
-   chore: bump minijinja from 2.7.0 to 2.8.0 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1405
-   chore: bump [@&#8203;playwright/test](https://github.com/playwright/test) from 1.50.1 to 1.51.0 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1401
-   chore: bump [@&#8203;types/node](https://github.com/types/node) from 22.13.8 to 22.13.10 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1400
-   chore: bump serde from 1.0.218 to 1.0.219 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1403
-   chore: bump serde_json from 1.0.139 to 1.0.140 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1404
-   chore: update dtolnay/rust-toolchain requirement to [`38b7019`](quarylabs/sqruff@38b7019) by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1397
-   chore: bump actions/setup-go from 5.1.0 to 5.3.0 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1396
-   chore: remove unused eslint config by [@&#8203;benfdking](https://github.com/benfdking) in quarylabs/sqruff#1408
-   chore: bump pyo3 from 0.23.5 to 0.24.0 by [@&#8203;dependabot](https://github.com/dependabot) in quarylabs/sqruff#1407
-   chore: finish lexer implementation by [@&#8203;benfdking](https://github.com/benfdking) in quarylabs/sqruff#1409
-   chore: bump version by [@&#8203;benfdking](https://github.com/benfdking) in quarylabs/sqruff#1410

**Full Changelog**: quarylabs/sqruff@v0.25.14...v0.25.15

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever MR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xOTIuMCIsInVwZGF0ZWRJblZlciI6IjM5LjE5Mi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
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.

[Bug]: The lexer breaks if the placeholder is within a comment or a quoted literal
1 participant