Merge pull request #11 from nlopes/nlopes-bump-to-0.2.0

Bump to 0.2.0

.github/workflows/ci.yml

@@ -97,5 +97,13 @@ jobs:
       run: rustup run ${{ matrix.toolchain }} cargo test --features sqlite,sqlx,runtime-async-std,macros --doc --verbose
 
     - name: Run clippy
-      run: rustup run ${{ matrix.toolchain }} cargo clippy --features sqlite,sqlx,runtime-async-std,macros -- -Dwarnings
-
+      run: |
+        rustup run ${{ matrix.toolchain }} cargo clippy --features sqlite,sqlx,runtime-async-std,macros --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features postgres,sqlx,runtime-async-std,macros --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features mysql,sqlx,runtime-async-std,macros --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features sqlite,diesel --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features postgres,diesel --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features mysql,diesel --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features sqlite,diesel,runtime-async-std,macros --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features postgres,diesel,runtime-async-std,macros --all-targets -- -Dwarnings
+        rustup run ${{ matrix.toolchain }} cargo clippy --features mysql,diesel,runtime-async-std,macros --all-targets -- -Dwarnings

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "database-schema"
-version = "0.1.2"
+version = "0.2.0"
 edition = "2021"
 description = "A library to dump a database schema into a file in SQL format"
 documentation = "https://docs.rs/database-schema"
@@ -42,7 +42,7 @@ sqlx = { version = "0.7", features = ["migrate", "macros"], optional = true, def
 thiserror = "1"
 tokio = { version = "1", features = ["rt-multi-thread"], optional = true, default-features = false }
 tracing = { version = "0.1", default-features = false }
-url = { version = "2.1", optional = true }
+url = { version = "2.5", optional = true }
 
 [dev-dependencies]
 tokio = { version = "1", features = ["rt", "macros"], default-features = false }

src/mysql/mod.rs

@@ -77,15 +77,15 @@ fn extract_connect_options(connection_url: &str) -> Result<MySqlConnectOptions,
     let username = url.username();
     if !username.is_empty() {
         options = options.username(
-            &*percent_decode_str(username)
+            &percent_decode_str(username)
                 .decode_utf8()
                 .map_err(Error::UriConfigurationDecoding)?,
         );
     }
 
     if let Some(password) = url.password() {
         options = options.password(
-            &*percent_decode_str(password)
+            &percent_decode_str(password)
                 .decode_utf8()
                 .map_err(Error::UriConfigurationDecoding)?,
         );
@@ -167,7 +167,7 @@ mod tests {
     async fn test_write_structure_sql() -> Result<(), crate::error::Error> {
         let destination_path = std::env::temp_dir().join("sqlx-mysql-structure.sql");
         let migrations_path = std::path::PathBuf::from("./fixtures/sqlx/mysql/migrations");
-        let _ = super::write_structure_sql(
+        super::write_structure_sql(
             super::DEFAULT_CONNECTION_URL,
             migrations_path,
             &destination_path,
@@ -184,7 +184,7 @@ mod tests {
     async fn test_write_structure_sql() -> Result<(), crate::error::Error> {
         let destination_path = std::env::temp_dir().join("diesel-mysql-structure.sql");
         let migrations_path = std::path::PathBuf::from("./fixtures/diesel/mysql/migrations");
-        let _ = super::write_structure_sql(
+        super::write_structure_sql(
             super::DEFAULT_CONNECTION_URL,
             migrations_path,
             &destination_path,

src/mysql/options.rs

@@ -12,7 +12,7 @@ use std::str::FromStr;
 /// Options for controlling the desired security state of the connection to the MySQL server.
 ///
 /// It is used by the [`ssl_mode`](super::MySqlConnectOptions::ssl_mode) method.
-#[derive(Debug, Clone, Copy)]
+#[derive(Default, Debug, Clone, Copy)]
 pub enum MySqlSslMode {
     /// Establish an unencrypted connection.
     Disabled,
@@ -21,6 +21,7 @@ pub enum MySqlSslMode {
     /// back to an unencrypted connection if an encrypted connection cannot be established.
     ///
     /// This is the default if `ssl_mode` is not specified.
+    #[default]
     Preferred,
 
     /// Establish an encrypted connection if the server supports encrypted connections.
@@ -50,12 +51,6 @@ impl std::fmt::Display for MySqlSslMode {
     }
 }
 
-impl Default for MySqlSslMode {
-    fn default() -> Self {
-        MySqlSslMode::Preferred
-    }
-}
-
 impl FromStr for MySqlSslMode {
     type Err = String;
 
@@ -68,7 +63,7 @@ impl FromStr for MySqlSslMode {
             "verify_identity" => MySqlSslMode::VerifyIdentity,
 
             _ => {
-                return Err(format!("unknown value {s:?} for `ssl_mode`").into());
+                return Err(format!("unknown value {s:?} for `ssl_mode`"));
             }
         })
     }

src/postgres/mod.rs

@@ -71,7 +71,7 @@ mod tests {
     ) -> Result<(), crate::error::Error> {
         let destination_path = std::env::temp_dir().join(destination_filename.as_ref());
         let migrations_path = std::path::PathBuf::from(fixtures_path.as_ref()).join("migrations");
-        let _ = super::write_structure_sql(
+        super::write_structure_sql(
             super::DEFAULT_CONNECTION_URL,
             migrations_path,
             &destination_path,