Skip to content

Fix CI error message in CI #1333

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

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Fix CI error message in CI #1333

merged 1 commit into from
Jul 8, 2024

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Jul 8, 2024

An error started happening after I merged #1318

I believe it is a logical conflict with #1319

Here is an example failure: https://github.com/sqlparser-rs/sqlparser-rs/actions/runs/9838692665/job/27159117248?pr=1314

---- parse_trailing_comma stdout ----
thread 'parse_trailing_comma' panicked at tests/sqlparser_common.rs:8969:5:
assertion failed: `(left == right)`

Diff < left / right > :
 ParserError(
<    "Expected: a privilege keyword, found: ON",
>    "Expected a privilege keyword, found: ON",
 )

@alamb alamb merged commit 17e5c0c into apache:main Jul 8, 2024
10 checks passed
@alamb alamb deleted the alamb/fix_ci branch July 8, 2024 11:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant