+1 (binding)
I ran into the same issue with cargo publish but commented that out.
Thanks, Andrew.
On Thu, Dec 12, 2024 at 1:58 PM Andrew Lamb wrote:
> Thank you for sticking with it :pray:
>
> On Thu, Dec 12, 2024 at 2:17 PM L. C. Hsieh wrote:
>
> > Okay. Once commented out the `cargo publish
Thank you for sticking with it :pray:
On Thu, Dec 12, 2024 at 2:17 PM L. C. Hsieh wrote:
> Okay. Once commented out the `cargo publish` step, it is okay.
>
> +1 (binding)
>
> Verified on M3 Mac.
>
> Thanks Andrew.
>
> On Thu, Dec 12, 2024 at 11:00 AM Andrew Lamb wrote:
> >
> > Yes, unfortunatel
Okay. Once commented out the `cargo publish` step, it is okay.
+1 (binding)
Verified on M3 Mac.
Thanks Andrew.
On Thu, Dec 12, 2024 at 11:00 AM Andrew Lamb wrote:
>
> Yes, unfortunately that happens due to [1]
>
> I think we just need to adjust the verification script to not run `cargo
> publi
Yes, unfortunately that happens due to [1]
I think we just need to adjust the verification script to not run `cargo
publish`
[1]: https://github.com/apache/datafusion-sqlparser-rs/issues/1596
On Thu, Dec 12, 2024 at 1:10 PM L. C. Hsieh wrote:
> Ah, I didn't notice that this is RC2.
>
> But I g
Ah, I didn't notice that this is RC2.
But I got another issue when running `verify-release-candidate.sh` with RC2:
```
error: failed to prepare local package for uploading
Caused by:
failed to select a version for the requirement `sqlparser_derive = "^0.3.0"`
candidate versions found whi
Thanks L.C. I also saw that problem with RC1 but thought I fixed it in RC2
(the PR[1] is here)
Can you confirm you are seeing this with RC2 as well?
Andrew
[1]: https://github.com/apache/datafusion-sqlparser-rs/pull/1595
On Thu, Dec 12, 2024 at 10:08 AM L. C. Hsieh wrote:
> There are a few fo
There are a few format issues while running `verify-release-candidate.sh`:
```
+ cargo fmt --all -- --check
Diff in
/private/var/folders/26/jr5lms716817rm8_wsr3w7n8gn/T/datafusion-sqlparser-rs-0.53.0.X.JaesmyIQIo/apache-datafusion-sqlparser-rs-0.53.0/derive/src/lib.rs:18:
use proc_ma
