Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: Handle LSP server request formatting #394

Merged
merged 3 commits into from
Jun 18, 2024
Merged

feat: Handle LSP server request formatting #394

merged 3 commits into from
Jun 18, 2024

Conversation

gvozdvmozgu
Copy link
Collaborator

No description provided.

@github-actions GitHub Actions
Copy link

Benchmark for c64871c

Click to view benchmark
Test Base PR %
DepthMap::from_parent 209.0±3.42µs 210.2±3.02µs +0.57%
fix_complex_query 21.7±0.14ms 21.6±0.41ms -0.46%
parse_complex_query 268.2±5.89µs 267.8±7.22µs -0.15%
parse_expression_recursion 297.5±6.80µs 298.4±6.17µs +0.30%
parse_simple_query 255.3±8.23µs 254.4±7.08µs -0.35%

@gvozdvmozgu gvozdvmozgu marked this pull request as ready for review June 18, 2024 04:43
@github-actions GitHub Actions
Copy link

Benchmark for e55db00

Click to view benchmark
Test Base PR %
DepthMap::from_parent 210.2±2.43µs 210.2±4.27µs 0.00%
fix_complex_query 21.8±0.18ms 21.8±0.71ms 0.00%
parse_complex_query 266.0±6.50µs 270.3±7.24µs +1.62%
parse_expression_recursion 300.0±8.28µs 294.1±5.51µs -1.97%
parse_simple_query 260.0±6.60µs 256.8±9.83µs -1.23%

@benfdking benfdking added this pull request to the merge queue Jun 18, 2024
Merged via the queue into main with commit 7ddae5a Jun 18, 2024
11 checks passed
@benfdking benfdking deleted the feature/lsp-server-request-formatting branch June 18, 2024 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benfdking benfdking benfdking approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gvozdvmozgu @benfdking