Bump parsers and add failed-to-parse and CI-practice badges#20
Merged
Conversation
Update the benchmarked parsers to their latest versions: polyglot-sql 0.4.4 to 0.5.1, sqlglot-rust 0.10.0 to 0.10.1, and the git-tracked sqlparser-rs to its current commit (b376022). pg_query and orql had no new code, and turso_parser stays on 0.6.1 since the only newer release is a prerelease. Add the new release points (polyglot-sql 0.5.1, sqlglot-rust 0.10.1) to the time machine and fix the hardcoded current_version table so the snapshot labels match the linked code. Add a failed-to-parse badge to each parser page showing how many statements it rejected that it was expected to accept, summed across every dialect, read from the failure totals already in the bundle. Add three CI-practice badges per parser: cargo deny, cargo audit, and cargo mutants, mined from each repository CI. turso is the only parser running cargo deny (its licenses check only) and none run cargo audit or cargo mutants. Regenerate the benchmark snapshot and the time-machine history to reflect the new versions.
The coverage job has failed since the time machine landed because timemachine links several sqlglot-rust versions side by side, each exporting the same unmangled extern C FFI symbols, which collide at link time once tarpaulin disables dead-code stripping. timemachine has no unit tests of its own, so excluding it restores a green, meaningful coverage run with no loss.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #20 +/- ##
==========================================
- Coverage 46.53% 39.59% -6.95%
==========================================
Files 17 23 +6
Lines 1906 2546 +640
==========================================
+ Hits 887 1008 +121
- Misses 1019 1538 +519 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the benchmarked parsers to their latest versions (polyglot-sql 0.4.4 to 0.5.1, sqlglot-rust 0.10.0 to 0.10.1, and sqlparser-rs to its current git commit), wires the new release points into the time machine, and fixes the hardcoded version table so the snapshot labels match the linked code. pg_query and orql had no new code, and turso_parser stays on 0.6.1 because the only newer release is a prerelease.
Adds a failed-to-parse badge to each parser page showing how many statements it rejected that it was expected to accept, summed across every dialect, plus three CI-practice badges (cargo deny, cargo audit, cargo mutants) mined from each repository. As of this snapshot turso is the only parser running cargo deny, and only its licenses check, and none run cargo audit or cargo mutants.
Regenerates the benchmark snapshot and the time-machine history.