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.

Sign up for GitHub

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

Jump to bottom

Bump version of sqlx, sea-query, sea-schema, time and uuid #834

Merged
merged 9 commits into from Jul 4, 2022
Merged

Bump version of sqlx, sea-query, sea-schema, time and uuid #834

merged 9 commits into from Jul 4, 2022

Conversation

billy1624
Copy link
Member

@billy1624 billy1624 commented Jul 3, 2022 •
edited

PR Info

Breaking Changes

  • Upgrade time to ^0.3
  • Upgrade sqlx to ^0.6
  • Upgrade uuid to ^1
  • Upgrade sea-query to ^0.26
  • Upgrade sea-schema to ^0.9.2
  • Sqlite chrono::DateTime<FixedOffset> timezone fix launchbadge/sqlx#1618
    SQLite chrono::DateTime<FixedOffset> timezone fix
    • DateTime<FixedOffset> will be stored in SQLite with the correct timezone instead of always in UTC.
      This was flagged as a "potentially breaking change" since it changes how dates are sent to SQLite.

@billy1624 billy1624 self-assigned this Jul 3, 2022
@billy1624 billy1624 mentioned this pull request Jul 3, 2022
Closed
3 tasks
@billy1624 billy1624 marked this pull request as ready for review July 4, 2022 07:35
@billy1624 billy1624 requested a review from tyt2y3 July 4, 2022 07:35
@billy1624 billy1624 mentioned this pull request Jul 4, 2022
Merged
ikrivosheev
ikrivosheev reviewed Jul 4, 2022
View reviewed changes
Copy link
Member

@ikrivosheev ikrivosheev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@billy1624 thank you, small comments.

Cargo.toml Outdated Show resolved Hide resolved
Cargo.toml Show resolved Hide resolved
ikrivosheev
ikrivosheev approved these changes Jul 4, 2022
View reviewed changes
Copy link
Member

@ikrivosheev ikrivosheev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

tyt2y3
tyt2y3 approved these changes Jul 4, 2022
View reviewed changes
Copy link
Member

@tyt2y3 tyt2y3 left a comment •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thank you all

@tyt2y3 tyt2y3 merged commit d6831e5 into master Jul 4, 2022
@tyt2y3 tyt2y3 deleted the bump-dependencies branch July 4, 2022 17:05
@billy1624 billy1624 linked an issue Jul 5, 2022 that may be closed by this pull request
Support time 0.3.x #788
Closed
This was referenced Jul 5, 2022
Closed
Closed
@billy1624 billy1624 linked an issue Jul 5, 2022 that may be closed by this pull request
Bump Uuid version to 1.0.0 #689
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tyt2y3 tyt2y3 tyt2y3 approved these changes

@ikrivosheev ikrivosheev ikrivosheev approved these changes

Assignees

@billy1624 billy1624

Labels
None yet
Projects
SeaQL Dev Tracker
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade SQLx dependency to 0.6.x Support time 0.3.x Bump Uuid version to 1.0.0
3 participants
@billy1624 @tyt2y3 @ikrivosheev