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

ci: Add job to check that sqlx builds with its declared minimum dependencies #3340

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

iamjpotts
Copy link
Contributor

@iamjpotts iamjpotts commented Jul 11, 2024

Prevents future occurrences of situations similar to #3118, where a change was made to sqlx to resolve a deprecation warning, the change is not supported by the dependency version declared in Cargo.toml, and the lock file has a newer version than what is declared Cargo.toml.

New CI job forces a downgrade of all dependencies to their minimum versions (ignoring the committed lock file), and attempts to compile the crate and its tests.

In Cargo.toml, the chrono and log crates were both upgraded to the oldest version that allowed the sqlx crate to build. This did not impact the lock file, which has newer versions of both.

@iamjpotts iamjpotts marked this pull request as draft July 11, 2024 01:26
@iamjpotts iamjpotts force-pushed the 20240710-check-minimal-versions branch from 5083db0 to d6510d3 Compare July 11, 2024 01:50
…dencies

Signed-off-by: Joshua Potts <8704475+iamjpotts@users.noreply.github.com>
…n declared minimum versions

Signed-off-by: Joshua Potts <8704475+iamjpotts@users.noreply.github.com>
@iamjpotts iamjpotts force-pushed the 20240710-check-minimal-versions branch from d6510d3 to df27b1a Compare July 11, 2024 02:01
@iamjpotts iamjpotts marked this pull request as ready for review July 11, 2024 02:04
@abonander abonander merged commit d1f180f into launchbadge:main Jul 11, 2024
65 checks passed
@iamjpotts
Copy link
Contributor Author

Clarification: The final (merged) version of this PR did not attempt to compile tests, as that would have required generating a .sqlx folder which requires a live database.

@iamjpotts iamjpotts deleted the 20240710-check-minimal-versions branch July 11, 2024 19:04
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