making it allways run on pull requests for now

Docki is now using cdn for Revealjs

.github/workflows/docker_deploy.yml

@@ -1,12 +1,10 @@
-name: Docker
+name: Deploy Docker Image to Github Container Registry
 
 on:
     push:
         branches: ["main"]
         paths:
             - "Cargo.toml"
-    pull_request:
-        branches: ["main"]
 
 env:
     CARGO_TERM_COLOR: always

.github/workflows/rust.yml

@@ -1,29 +0,0 @@
-name: Rust
-
-on:
-  push:
-    branches: [ "main" ]
-    paths:
-      - 'Cargo.toml'
-  pull_request:
-    branches: [ "main" ]
-
-env:
-  CARGO_TERM_COLOR: always
-
-jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v4
-    - name: Build
-      run: cargo build --verbose
-    - name: Run tests
-      run: cargo test --verbose
-    - name: Publish to crates.io
-      env:
-        CARGO_REGISTRY_TOKEN: ${{ secrets.CRATES_IO_TOKEN }}
-      run: |
-        cargo publish --verbose

.github/workflows/rust_deploy.yml

@@ -0,0 +1,26 @@
+name: Deploy Rust to Crates.io
+
+on:
+    push:
+        branches: ["main"]
+        paths:
+            - "Cargo.toml"
+
+env:
+    CARGO_TERM_COLOR: always
+
+jobs:
+    build:
+        runs-on: ubuntu-latest
+
+        steps:
+            - uses: actions/checkout@v4
+            - name: Build
+              run: cargo build --verbose
+            - name: Run tests
+              run: cargo test --verbose
+            - name: Publish to crates.io
+              env:
+                  CARGO_REGISTRY_TOKEN: ${{ secrets.CRATES_IO_TOKEN }}
+              run: |
+                  cargo publish --verbose

.github/workflows/rust_test.yml

@@ -0,0 +1,24 @@
+name: Test Rust
+
+on:
+    push:
+        branches: ["main", "develop"]
+        paths:
+            - "./src/**"
+            - "./Cargo.toml"
+    pull_request:
+        branches: ["main", "develop"]
+
+env:
+    CARGO_TERM_COLOR: always
+
+jobs:
+    build:
+        runs-on: ubuntu-latest
+
+        steps:
+            - uses: actions/checkout@v4
+            - name: Build
+              run: cargo build --verbose
+            - name: Run tests
+              run: cargo test --verbose

Cargo.lock

@@ -630,17 +630,6 @@ dependencies = [
  "unicode-width",
 ]
 
-[[package]]
-name = "colored"
-version = "2.0.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b3616f750b84d8f0de8a58bda93e08e2a81ad3f523089b05f1dffecab48c6cbd"
-dependencies = [
- "atty",
- "lazy_static",
- "winapi 0.3.9",
-]
-
 [[package]]
 name = "concurrent-queue"
 version = "2.1.0"
@@ -859,16 +848,16 @@ checksum = "212d0f5754cb6769937f4501cc0e67f4f4483c8d2c3e1e922ee9edbe4ab4c7c0"
 
 [[package]]
 name = "docki"
-version = "1.2.2"
+version = "1.2.3"
 dependencies = [
  "bytes",
  "clap 4.1.8",
  "clap_complete",
- "colored",
  "futures",
  "home",
  "live-server",
  "notify 5.1.0",
+ "nu-ansi-term",
  "regex",
  "reqwest",
  "text_io",
@@ -1872,6 +1861,15 @@ dependencies = [
  "windows-sys 0.42.0",
 ]
 
+[[package]]
+name = "nu-ansi-term"
+version = "0.50.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "7957b9740744892f114936ab4a57b3f487491bbeafaf8083688b16841a4240e5"
+dependencies = [
+ "windows-sys 0.61.2",
+]
+
 [[package]]
 name = "num-integer"
 version = "0.1.45"
@@ -3206,6 +3204,12 @@ version = "0.4.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "712e227841d057c1ee1cd2fb22fa7e5a5461ae8e48fa2ca79ec42cfc1931183f"
 
+[[package]]
+name = "windows-link"
+version = "0.2.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "f0805222e57f7521d6a62e36fa9163bc891acd422f971defe97d64e70d0a4fe5"
+
 [[package]]
 name = "windows-sys"
 version = "0.42.0"
@@ -3230,6 +3234,15 @@ dependencies = [
  "windows-targets",
 ]
 
+[[package]]
+name = "windows-sys"
+version = "0.61.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "ae137229bcbd6cdf0f7b80a31df61766145077ddf49416a728b02cb3921ff3fc"
+dependencies = [
+ "windows-link",
+]
+
 [[package]]
 name = "windows-targets"
 version = "0.42.1"

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "docki"
-version = "1.2.2"
+version = "1.2.3"
 edition = "2021"
 description = "cli for building and publishing documentation using asciidoctor"
 license-file = "LICENSE.txt"
@@ -12,7 +12,6 @@ exclude = [".gitlab", ".github"]
 [dependencies]
 bytes = "1.4.0"
 clap = { version = "4.1.8", features = ["derive"] }
-colored = "2.0.0"
 futures = "0.3.26"
 home = "0.5.4"
 live-server = "0.6.0"
@@ -23,3 +22,4 @@ text_io = "0.1.12"
 tokio = { version = "1.26.0", features = ["full"] }
 zip-extract = "0.1.1"
 clap_complete = "4.1.4"
+nu-ansi-term = "0.50.3"

src/app/args/structure.rs

@@ -16,10 +16,15 @@ pub enum ShellArg {
 #[derive(Subcommand)]
 pub enum CommandArg {
     /// Builds the documentation into a dist folder
-    Build,
+    Build {
+		/// When set to true, docki will download revealjs before building the documentation.
+		/// Otherwise it will use the cdn for revealjs
+        #[arg(short, long)]
+        offline_reveal: bool,
+    },
     /// Checks if everything required for docki is installed
     Health,
-    /// Helper command for installing asciidoctor-reveal-js
+    /// Deprecated: Helper command for installing asciidoctor-reveal-js
     InstallReveal,
     /// Starts a Webserver with the live preview of the Documentation
     Serve {

src/app/build/asciidoctor.rs

@@ -1,3 +1,4 @@
+use regex::Regex;
 use std::process;
 
 fn exec_command(command: &mut process::Command) -> Result<(), String> {
@@ -27,9 +28,14 @@ fn asciidoctor_docs(in_path: &str, out_path: &str) -> process::Command {
     return command;
 }
 
-fn asciidoctor_slides(in_path: &str, out_path: &str) -> process::Command {
+fn asciidoctor_slides(in_path: &str, out_path: &str, offline_reveal: bool) -> process::Command {
     let mut command = process::Command::new(format!("asciidoctor-revealjs"));
-    let revealjs_path = "/slides/revealjs";
+    let out_dir = parent_path(out_path);
+	let revealjs_path = if offline_reveal {
+		path_between(out_dir.to_string(), "./dist/slides/revealjs".to_string())
+	} else {
+		"https://cdn.jsdelivr.net/npm/reveal.js@5.2.1".to_string()
+	};
 
     command
         .arg(format!("{in_path}"))
@@ -40,13 +46,67 @@ fn asciidoctor_slides(in_path: &str, out_path: &str) -> process::Command {
     return command;
 }
 
+fn parent_path(child_path: &str) -> String {
+    let split: Vec<&str> = child_path.split("/").collect();
+    let slice = &split[..split.len() - 1];
+    return slice.join("/");
+}
+
+pub fn path_between(from: String, to: String) -> String {
+    let from_segments = transform_input_to_clone_split(&from);
+    let to_segments = transform_input_to_clone_split(&to);
+    let last_matching_index = matching_from_start(&from_segments, &to_segments);
+    let number_of_backs = from_segments.len() - last_matching_index;
+    let mut path_between = path_back(number_of_backs);
+    let path_to_to_path = &to_segments[last_matching_index..];
+    path_between.push_str(&path_to_to_path.join("/"));
+    return path_between;
+}
+
+fn transform_input_to_clone_split(input: &String) -> Vec<String> {
+    let regex = Regex::new(r"/$").unwrap();
+    let first_transformation = input.clone().replace("./", "");
+    return regex
+        .replace_all(&first_transformation, "")
+        .to_string()
+        .split("/")
+        .collect::<Vec<&str>>()
+        .iter()
+        .map(|s| s.to_string())
+        .collect();
+}
+
+fn path_back(count: usize) -> String {
+    let mut path = "".to_string();
+
+    for _ in 0..count {
+        path.push_str("../");
+    }
+
+    return path;
+}
+
+pub fn matching_from_start(from_segments: &Vec<String>, to_segments: &Vec<String>) -> usize {
+    for (index, from_segment) in from_segments.iter().enumerate() {
+        if let Some(to_segment) = to_segments.get(index) {
+            if from_segment != to_segment {
+                return index;
+            }
+        } else {
+            return index;
+        }
+    }
+
+    return from_segments.len();
+}
+
 pub fn build_doc(in_path: &str, out_path: &str) -> Result<(), String> {
     let mut command = asciidoctor_docs(in_path, out_path);
     return exec_command(&mut command);
 }
 
-pub fn build_slide(in_path: &str, out_path: &str) -> Result<(), String> {
+pub fn build_slide(in_path: &str, out_path: &str, offline_reveal: bool) -> Result<(), String> {
-    let mut command = asciidoctor_slides(in_path, out_path);
+    let mut command = asciidoctor_slides(in_path, out_path, offline_reveal);
     return exec_command(&mut command);
 }
 

src/app/build/mod.rs

@@ -6,12 +6,12 @@ use super::fs_util;
 
 pub mod asciidoctor;
 
-pub fn docki_build(in_path: &str) -> DockiBuildResult {
+pub fn docki_build(in_path: &str, offline_reveal: bool) -> DockiBuildResult {
     let out_path = in_path.replace("/docs/", "/dist/");
     let convert_out_path = out_path.replace(".adoc", ".html");
 
     if in_path.starts_with("./docs/slides/") && in_path.ends_with(".adoc") {
-        if let Err(err) = build_slide(&in_path, &convert_out_path) {
+        if let Err(err) = build_slide(&in_path, &convert_out_path, offline_reveal) {
             return DockiBuildResult::Err(err);
         }
 
@@ -23,6 +23,10 @@ pub fn docki_build(in_path: &str) -> DockiBuildResult {
 
         DockiBuildResult::Doc(convert_out_path)
     } else {
+		if in_path.starts_with("./docs/slides/revealjs") && !offline_reveal {
+			return DockiBuildResult::Silent;
+		}
+
         if let Err(err) = copy(&in_path, &out_path) {
             return DockiBuildResult::Err(err);
         }
@@ -46,4 +50,5 @@ pub enum DockiBuildResult {
     Doc(String),
     Copy(String),
     Err(String),
+	Silent,
 }

src/app/commands/build.rs

@@ -1,6 +1,6 @@
 use super::executions::build_execution::BuildExecution;
 
-pub async fn build() -> () {
+pub async fn build(offline_reveal: bool) -> () {
     let mut build_execution = BuildExecution::new();
-    build_execution.execute().await.expect("build failed")
+    build_execution.execute(offline_reveal).await.expect("build failed")
 }

src/app/commands/executions/build_execution.rs

@@ -21,7 +21,7 @@ impl BuildExecution {
         };
     }
 
-    pub async fn execute(&mut self) -> Result<(), String> {
+    pub async fn execute(&mut self, offline_reveal: bool) -> Result<(), String> {
         let path = "./docs/".to_string();
 
         if !fs_util::directory_exists(&path) {
@@ -30,21 +30,25 @@ impl BuildExecution {
             );
         }
 
-        if let Err(error) = Self::prepare().await {
+        if let Err(error) = Self::prepare(offline_reveal).await {
             return Err(error);
         }
 
-        return self.build_dir(&path);
+        return self.build_dir(&path, offline_reveal);
     }
 
 
-    async fn prepare() -> Result<(), String> {
+    async fn prepare(offline_reveal: bool) -> Result<(), String> {
+		if !offline_reveal {
+			return Ok(())
+		}
+
         let reveal_version = "5.2.1";
         let target = format!("https://github.com/hakimel/reveal.js/archive/{reveal_version}.zip");
 
         create_dir_recursive("./docs/slides");
 
-		let response = reqwest::get(target.clone()).await.unwrap();
+		reqwest::get(target.clone()).await.unwrap();
         let Ok(response) = reqwest::get(target).await else {
             return Err("could not downlaod revealjs".to_string())
         };
@@ -62,7 +66,7 @@ impl BuildExecution {
         return Ok(());
     }
 
-    fn build_dir(&mut self, path: &str) -> Result<(), String> {
+    fn build_dir(&mut self, path: &str, offline_reveal: bool) -> Result<(), String> {
         let result = fs_util::fetch_paths_recursive(&path);
 
         let Ok(paths) = result else {
@@ -72,7 +76,7 @@ impl BuildExecution {
         for (index, in_path) in paths.iter().enumerate() {
             self.progress = index + 1;
             self.goal = paths.len();
-            let result = docki_build(&in_path);
+            let result = docki_build(&in_path, offline_reveal);
 
             match result {
                 DockiBuildResult::Err(err) => {
@@ -81,7 +85,8 @@ impl BuildExecution {
                 },
                 DockiBuildResult::Copy(out_path) => self.display_building_status("Copy", &in_path, &out_path),
                 DockiBuildResult::Slide(out_path) => self.display_building_status("Slide", &in_path, &out_path),
-                DockiBuildResult::Doc(out_path) => self.display_building_status("Doc", &in_path, &out_path)
+                DockiBuildResult::Doc(out_path) => self.display_building_status("Doc", &in_path, &out_path),
+				DockiBuildResult::Silent => ()
             }
         }
 

src/app/commands/health.rs

@@ -1,5 +1,5 @@
-use std::{process::Command, io::ErrorKind};
+use nu_ansi_term::Color::{LightGray, LightGreen, LightRed};
-use colored::Colorize;
+use std::{io::ErrorKind, process::Command};
 
 const INFO_ASCIIDOC: &str = "
 Install the binary with your package manager!
@@ -37,7 +37,7 @@ fn check_reveal() -> () {
 }
 
 fn reveal_is_installed() -> bool {
-    return check_command("asciidoctor-revealjs")
+    return check_command("asciidoctor-revealjs");
 }
 
 fn check_asciidoc() -> () {
@@ -49,23 +49,21 @@ fn check_asciidoc() -> () {
 }
 
 fn asciidoc_is_installed() -> bool {
-    return check_command("asciidoctor")
+    return check_command("asciidoctor");
 }
 
 fn check_command(command: &str) -> bool {
-    return match Command::new(command)
+    return match Command::new(command).output() {
-        .output() {
         Ok(_) => true,
-        Err(e) => ErrorKind::NotFound != e.kind()
+        Err(e) => ErrorKind::NotFound != e.kind(),
-    }
+    };
 }
 
 fn print_health_ok(name: &str) {
-    println!("- ✔️ {}", name.bright_green());
+    println!("- ✔️ {}", LightGreen.paint(name));
 }
 
 fn print_health_not_ok(name: &str, info: &str) {
-    println!("- ❗{}", name.bright_red());
+    println!("- ❗{}", LightRed.paint(name));
-    println!("{}", info.bright_black())
+    println!("{}", LightGray.paint(info))
 }
-

src/app/commands/serve.rs

@@ -1,4 +1,4 @@
-use colored::Colorize;
+use nu_ansi_term::Color::Green;
 use futures::StreamExt;
 use live_server::listen;
 use notify::{
@@ -11,7 +11,7 @@ use crate::app::{ watcher::watcher, build::{docki_build, DockiBuildResult}, comm
 
 
 pub async fn serve(port: Option<u16>) {
-    build().await;
+    build(false).await;
     tokio::join!(watch_and_build(), start_server(port));
 }
 
@@ -21,15 +21,18 @@ async fn watch_and_build() {
         .expect("something went wrong")
 }
 
+
 async fn start_server(port: Option<u16>) {
-    let unwrapped_port = port.unwrap_or(8080);
+    let port = port.unwrap_or(8080);
+	let link = &format!("http://localhost:{}", port);
+	let hyperlink = Green.paint(link).hyperlink(link);
+
     println!(
-        "\nServing at {}{} ",
+        "\nServing at {}",
-        "http://localhost:".bold(),
+        hyperlink
-        unwrapped_port.to_string().bold()
     );
 
-    let Ok(()) = listen("localhost", port.unwrap_or(8080), "./dist").await else {
+    let Ok(()) = listen("localhost", port, "./dist").await else {
         panic!("could not start server")
     };
 }
@@ -65,7 +68,7 @@ fn build_file(paths: Vec<std::path::PathBuf>) {
         .expect(invalid_path_message);
 
 	let in_path = format!("./{}", in_path);
-    let result = docki_build(&in_path);
+    let result = docki_build(&in_path, false);
 
     match result {
         DockiBuildResult::Slide(out_path) => display_rebuilding_status("Slide", &in_path, &out_path),
@@ -75,6 +78,7 @@ fn build_file(paths: Vec<std::path::PathBuf>) {
             display_rebuilding_status("Error", &in_path, "");
             println!("{}", err);
         },
+		DockiBuildResult::Silent => ()
     }
 }
 

src/app/log/mod.rs

@@ -1,20 +1,19 @@
-use colored::Colorize;
+use nu_ansi_term::Color::LightGreen;
+use nu_ansi_term::Color::LightRed;
+use nu_ansi_term::Style;
 
 pub fn display_status(context1: &str, context2: &str, in_path: &str, out_path: &str) {
-    let colored_context = color_context(context2);
+    let colored_context = if context2 == "Error" {
+        LightRed.paint(context2)
+    } else {
+        LightGreen.paint(context2)
+    };
+
     println!(
         "({}) [{}] {} -> {}",
-        context1.bold(),
+        Style::new().paint(context1),
         colored_context,
         in_path,
         out_path
     );
 }
-
-fn color_context(context: &str) -> colored::ColoredString {
-    if context == "Error" {
-        return context.bright_red()
-    } else {
-        return context.bright_green()
-    }
-}

src/app/mod.rs

@@ -23,7 +23,7 @@ impl App {
         Self::setup_environment_variables();
 
         match args.command {
-            CommandArg::Build => build().await,
+            CommandArg::Build { offline_reveal } => build(offline_reveal).await,
             CommandArg::Health => health(),
             CommandArg::InstallReveal => install_reveal().await,
             CommandArg::Serve { port } => serve(port).await,