Remove redundant normalize_url()

Signed-off-by: Austin Liu <[email protected]>
PyO3 · Sep 2, 2024 · fa5a470 · fa5a470
1 parent e48c35a
commit fa5a470
Showing 1 changed file with 4 additions and 12 deletions.
diff --git a/src/upload.rs b/src/upload.rs
@@ -59,8 +59,8 @@ pub struct PublishOpt {
 }
 
 impl PublishOpt {
-    const DEFAULT_REPOSITORY_URL: &'static str = "https://upload.pypi.org/legacy/";
-    const TEST_REPOSITORY_URL: &'static str = "https://test.pypi.org/legacy/";
+    const DEFAULT_REPOSITORY_URL: &'static str = "https://upload.pypi.org/legacy";
+    const TEST_REPOSITORY_URL: &'static str = "https://test.pypi.org/legacy";
 
     /// Set to non interactive mode if we're running on CI
     pub fn non_interactive_on_ci(&mut self) {
@@ -322,16 +322,8 @@ fn complete_registry(opt: &PublishOpt) -> Result<Registry> {
     {
         let normalized_url = PublishOpt::normalize_url(repository_url);
         let name = match normalized_url {
-            normalized
-                if normalized == PublishOpt::normalize_url(PublishOpt::DEFAULT_REPOSITORY_URL) =>
-            {
-                Some("pypi")
-            }
-            normalized
-                if normalized == PublishOpt::normalize_url(PublishOpt::TEST_REPOSITORY_URL) =>
-            {
-                Some("testpypi")
-            }
+            normalized if normalized == PublishOpt::DEFAULT_REPOSITORY_URL => Some("pypi"),
+            normalized if normalized == PublishOpt::TEST_REPOSITORY_URL => Some("testpypi"),
             _ => None,
         };
         (name, repository_url.to_string())