Skip to content

refactor(sqlite): Resolve duplicate test target warning for macros.rs #3988

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

iamjpotts
Copy link
Contributor

Does your PR solve an issue?

Resolves this cargo warning:

warning: ./Cargo.toml: file `./tests/sqlite/macros.rs` found to be present in multiple build targets:
  * `integration-test` target `sqlite-macros`
  * `integration-test` target `sqlite-unbundled-macros`

Merges the two target definitions in Cargo.toml and moves the sqlite vs sqlite-unbundled feature check to the top of the macros.rs module.

Is this a breaking change?

No; only affects test target configuration.

@iamjpotts iamjpotts force-pushed the jp/sqlite-macros-integration-test-targets branch from 93a1dbd to f1ee6df Compare August 22, 2025 03:52
@iamjpotts iamjpotts marked this pull request as ready for review August 22, 2025 04:06
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.

1 participant