Skip to content

Conversation

@vimko
Copy link
Contributor

@vimko vimko commented May 9, 2025

According to statistics, Python 3.7's current usage share still exceeds 20%. Since sqloxide itself supports Python 3.7, we hope to ensure support for it when building wheels.

vimko added 2 commits May 9, 2025 12:22
According to statistics, Python 3.7's current usage share still exceeds 20%. Since sqloxide itself supports Python 3.7, we hope to ensure support for it when building wheels.
Notes:

In v0.55 of sqlparser-rs, the `ObjectName` structure has been changed as shown below. Here is now to migrate.

```diff
- pub struct ObjectName(pub Vec<Ident>);
+ pub struct ObjectName(pub Vec<ObjectNamePart>)
```

Therefore, when using the `parse_sql` function, the data structure of the table name in the return value will change.

Previously:

```json
{
    "value": "employee",
    "quote_style": null,
    "span":
    {
        "start":
        {
            "line": 4,
            "column": 10
        },
        "end":
        {
            "line": 4,
            "column": 18
        }
    }
}
```

Now:

```json
{
    "Identifier":
    {
        "value": "employee",
        "quote_style": null,
        "span":
        {
            "start":
            {
                "line": 4,
                "column": 10
            },
            "end":
            {
                "line": 4,
                "column": 18
            }
        }
    }
}
```
@vimko vimko changed the title Update build.yaml to support build wheels for python 3.7 Update build.yaml to support build wheels for python 3.7 and Upgrade to sqlparser-rs 0.56.0 May 17, 2025
@vimko
Copy link
Contributor Author

vimko commented May 17, 2025

@wseaton review pls.

@vimko vimko marked this pull request as draft May 17, 2025 06:54
@vimko vimko marked this pull request as ready for review May 17, 2025 07:56
@wseaton
Copy link
Owner

wseaton commented May 17, 2025

Thanks, I will review today!

@wseaton wseaton merged commit 407d42c into wseaton:master May 17, 2025
@wseaton
Copy link
Owner

wseaton commented May 17, 2025

@vimko thank you for the help here, I have cut a release!

@vimko
Copy link
Contributor Author

vimko commented May 19, 2025

@wseaton Thank you very much for developing such a useful library!

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.

2 participants