feat: add --tag option as an alias of --branch

Author: yoannfleurydev

CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.1.13] - 2020-06-20
+
+### Added
+
+- add support for `--commit` to open a specific commit by [@rubenrua](https://github.com/rubenrua)
+- add alias `--tag` for `--branch` as they are the same from git host provider perspective
+
 ## [0.1.12] - 2020-05-16
 
 ### Changed

CONTRIBUTING.md

@@ -1,5 +1,9 @@
 # CONTRIBUTING
 
+# Pull Request
+
+Just do the changes, we'll discuss about it on the PR.
+
 ## PUBLISHING
 
 - [ ] Change the code you want to update.

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "gitweb"
-version = "0.1.12"
+version = "0.1.13"
 authors = ["Yoann Fleury <yoann.fleury@yahoo.com>"]
 edition = "2018"
 description = "Open the current remote repository in your browser"

Cargo.toml

@@ -3,9 +3,12 @@ use git2::{Error, ErrorCode, Repository};
 
 /// Get the current repository.
 pub fn get_repo() -> Result<Repository, Error> {
-    Repository::discover(".")
+    const CURRENT_WORKING_DIRECTORY: &str = ".";
+
+    Repository::discover(CURRENT_WORKING_DIRECTORY)
 }
 
+// Get the current branch or return master.
 pub fn get_branch(repo: &Repository, logger: &Logger) -> String {
     let head = match repo.head() {
         Ok(head) => Some(head),

src/git.rs

@@ -59,15 +59,19 @@ fn main() {
         }
     };
 
-    // Get the branch to show in the browser. If the option is given, then, the
-    // value will be used, else, the current branch is given, or master if
-    // something went wrong.
-    let branch = match opt.branch {
-        Some(branch) => branch,
-        None => {
-            logger.verbose_print("No branch given, getting current one");
-
-            git::get_branch(&repo, &logger)
+    // Get the tag to show in the browser. If the option is given, then the value
+    // will be used as it is an alias for branch.
+    let reference = if let Some(tag) = opt.tag { tag } else {
+        // Get the branch to show in the browser. If the option is given, then, the
+        // value will be used, else, the current branch is given, or master if
+        // something went wrong.
+        match opt.branch {
+            Some(branch) => branch,
+            None => {
+                logger.verbose_print("No branch given, getting current one");
+
+                git::get_branch(&repo, &logger)
+            }
         }
     };
 
@@ -105,11 +109,11 @@ fn main() {
         )
     } else {
         format!(
-            "https://{domain}/{repository}/{path}/{branch}",
+            "https://{domain}/{repository}/{path}/{reference}",
             domain = parts.0,
             path = if parts.0 == BITBUCKET_HOSTNAME {"src"} else {"tree"},
             repository = parts.1,
-            branch = branch
+            reference = reference
         )
     };
 

src/main.rs

@@ -6,14 +6,22 @@ use structopt::StructOpt;
 // Rename all will use the name of the field
 #[structopt(rename_all = "kebab-case")]
 pub struct Opt {
-    /// Set the branch
+    /// Set the branch (alias for --tag)
     ///
     /// By setting the branch, you can override the default behavior that will
     /// set the branch to the current one in the repository. If something went
     /// wrong with the current one, it will set the value to master.
     #[structopt(short, long)]
     pub branch: Option<String>,
 
+    /// Set the tag (alias for --branch).
+    ///
+    /// By setting the tag, you can override the default behavior that will set
+    /// the branch to the current one in the repository and will instead take
+    /// the reference tag (or branch) given as parameter.
+    #[structopt(short, long)]
+    pub tag: Option<String>,
+
     /// Set a commit
     ///
     /// By setting a commit, you can override the default behavior that will