Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,5 @@ edition = "2024"

[dependencies]
clap = { version = "4.5", features = ["derive", "env"] }
anyhow = "1"

65 changes: 65 additions & 0 deletions src/command/analyze.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
use anyhow::bail;

#[derive(Debug, PartialEq)]
pub struct ParsedRepo {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think Repo is enough but can be changed in another PR

pub org: String,
pub repo: String,
}

pub fn parse_repo(input: &str) -> anyhow::Result<ParsedRepo> {
let parts: Vec<&str> = input.split('/').collect();
if parts.len() != 2 || parts[0].is_empty() || parts[1].is_empty() {
bail!("--repo must be in the form owner/name");
}
Ok(ParsedRepo {
org: parts[0].to_string(),
repo: parts[1].to_string(),
})
}

pub fn run(repo_arg: Option<String>, org_arg: Option<String>) -> anyhow::Result<()> {
if let Some(repo_arg) = repo_arg {
let parsed = parse_repo(&repo_arg)?;
println!("parsed org={} repo={}", parsed.org, parsed.repo);
} else if org_arg.is_some() {
bail!("--org is not yet supported");
}
Ok(())
}

#[cfg(test)]
mod tests {
use super::*;

#[test]
fn parses_valid_input() {
let parsed = parse_repo("rust-lang/crabwatch").unwrap();
assert_eq!(
parsed,
ParsedRepo {
org: "rust-lang".to_string(),
repo: "crabwatch".to_string(),
}
);
}

#[test]
fn rejects_input_without_slash() {
assert!(parse_repo("rust-lang").is_err());
}

#[test]
fn rejects_empty_owner() {
assert!(parse_repo("/crabwatch").is_err());
}

#[test]
fn rejects_empty_name() {
assert!(parse_repo("rust-lang/").is_err());
}

#[test]
fn rejects_too_many_parts() {
assert!(parse_repo("a/b/c").is_err());
}
}
1 change: 1 addition & 0 deletions src/command/mod.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
pub mod analyze;
13 changes: 10 additions & 3 deletions src/main.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
use clap::{ArgGroup, Parser, Subcommand};
use std::path::PathBuf;

mod command;

/// Analyze CI and best practices across Rust project repos
#[derive(Parser)]
#[command(name = "crabwatch", version, about, long_about = None)]
Expand Down Expand Up @@ -40,11 +42,16 @@ enum Command {
},
}

fn main() {
fn main() -> anyhow::Result<()> {
let cli = Cli::parse();
match cli.command {
Command::Analyze { repo, org, check } => {
println!("analyze repo={repo:?} org={org:?} check={check:?}");
Command::Analyze {
repo,
org,
check: _,
} => {
command::analyze::run(repo, org)?;
}
}
Ok(())
}