Skip to content
GitLab
Explore
Sign in
Register
Primary navigation
Search or go to…
Project
S
sbroad
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Package Registry
Container Registry
Model registry
Operate
Environments
Terraform modules
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
core
sbroad
Merge requests
!435
fix: parsing count() panics
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
fix: parsing count() panics
fix_count_parsing_bug
into
main
Overview
0
Commits
1
Pipelines
1
Changes
2
Merged
Arseniy Volynets
requested to merge
fix_count_parsing_bug
into
main
10 months ago
Overview
0
Commits
1
Pipelines
1
Changes
2
Expand
Summarize the changes
fix: parsing panics
Ensure that
New code is covered by unit and integration tests.
Related issues would be automatically closed with gitlab's closing pattern (
Closes #1, #2
).
Public modules are documented (check the rendered version with
cargo doc --open
).
(if PEST grammar is changed) EBNF grammar reflects these changes (check the result with railroad diagram
generator
.
closes
#646 (closed)
Next steps
Update sbroad submodule in
picodata/picodata
.
(if EBNF grammar is changed) create a follow-up issue in
picodata/docs
.
Edited
10 months ago
by
Arseniy Volynets
0
0
Merge request reports
Activity
All activity
Filter activity
Deselect all
Approvals
Assignees & reviewers
Comments (from bots)
Comments (from users)
Commits & branches
Edits
Labels
Lock status
Mentions
Merge request status
Tracking
Loading
Loading
Loading
Loading
Loading
Please
register
or
sign in
to reply
Loading