Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove dependency on quickcheck_macros #245

Merged
merged 3 commits into from
Sep 12, 2023
Merged

Remove dependency on quickcheck_macros #245

merged 3 commits into from
Sep 12, 2023

Conversation

lopopolo
Copy link
Member

The macro_rules macro quickcheck::quickcheck is the preferred way to use quickcheck in tests. The crate author is not a user of the syn-based quickcheck_macros attribute macro. This dep has atrophied and has not been updated to syn v2.

Remove the attribute macro and replace its use with the equivalent macro_rules macro. Update minimum bound on version-sync to v0.9.5.

These changes combine to fix the duplicate crate warnings found by cargo-deny on trunk:

error[duplicate]: found 2 duplicate entries for crate 'syn'
   ┌─ /github/workspace/Cargo.lock:23:1
   │
23 │ ╭ syn 1.0.109 registry+https://github.com/rust-lang/crates.io-index
24 │ │ syn 2.0.32 registry+https://github.com/rust-lang/crates.io-index
   │ ╰────────────────────────────────────────────────────────────────^ lock entries
   │
   = syn v1.0.109
     └── quickcheck_macros v1.0.0
         └── (dev) intaglio v1.9.1
   = syn v2.0.32
     └── version-sync v0.9.5
         └── (dev) intaglio v1.9.1

Fixes build failure in https://github.com/artichoke/intaglio/actions/runs/6153154527/job/16696558548.

This diff is best viewed when ignoring whitespace.

@lopopolo lopopolo added the A-deps Area: Source and library dependencies. label Sep 12, 2023
The `macro_rules` macro `quickcheck::quickcheck` is the preferred way to
use `quickcheck` in tests. The crate author is not a user of the
`syn`-based `quickcheck_macros` attribute macro. This dep has atrophied
and has not been updated to `syn` v2.

Remove the attribute macro and replace its use with the equivalent
`macro_rules` macro. Update minimum bound on `version-sync` to v0.9.5.

These changes combine to fix the duplicate crate warnings found by
`cargo-deny` on trunk:

```
error[duplicate]: found 2 duplicate entries for crate 'syn'
   ┌─ /github/workspace/Cargo.lock:23:1
   │
23 │ ╭ syn 1.0.109 registry+https://github.com/rust-lang/crates.io-index
24 │ │ syn 2.0.32 registry+https://github.com/rust-lang/crates.io-index
   │ ╰────────────────────────────────────────────────────────────────^ lock entries
   │
   = syn v1.0.109
     └── quickcheck_macros v1.0.0
         └── (dev) intaglio v1.9.1
   = syn v2.0.32
     └── version-sync v0.9.5
         └── (dev) intaglio v1.9.1
```
@lopopolo lopopolo merged commit 425644b into trunk Sep 12, 2023
@lopopolo lopopolo deleted the lopopolo/syn-v2 branch September 12, 2023 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-deps Area: Source and library dependencies.
Development

Successfully merging this pull request may close these issues.

1 participant