3 Commits

Author SHA1 Message Date
dc0040c2d1 Add workflow to check Go code on push and pull requests 2025-10-24 23:34:15 +02:00
9950282d6e Update build-and-release workflow to use Gitea-hosted setup-go action
Some checks failed
Release binary for cl-cli / build-and-release (push) Failing after 7s
2025-10-24 23:33:31 +02:00
7c8e8eb236 Clean code by commenting out some variable, and allowing some dead code. 2025-10-24 23:24:47 +02:00
10 changed files with 45 additions and 45 deletions

View File

@@ -21,7 +21,7 @@ jobs:
with: with:
path: .changes/${{ github.ref_name }}.md path: .changes/${{ github.ref_name }}.md
- name: Setup go for using go gitea actions - name: Setup go for using go gitea actions
uses: https://github.com/actions/setup-go@v4 uses: https://gitea.com/actions/setup-go@v4
with: with:
go-version: '>=1.20.1' go-version: '>=1.20.1'
- name: Use Go Action to release - name: Use Go Action to release

View File

@@ -0,0 +1,18 @@
name: Check go code
on:
push: *
pull_request: *
jobs:
build-and-release:
runs-on: ubuntu-latest
steps:
- name: Checkout repository
uses: https://github.com/actions/checkout@v4
- name: Install rust toolchain
uses: https://github.com/dtolnay/rust-toolchain@stable
- name: Build binaries
run: cargo build
- name: Run tests
run: cargo test

View File

@@ -7,7 +7,7 @@ pub struct GeneralError {
} }
impl From<InvalidHeaderValue> for GeneralError { impl From<InvalidHeaderValue> for GeneralError {
fn from(value: InvalidHeaderValue) -> Self { fn from(_value: InvalidHeaderValue) -> Self {
GeneralError { GeneralError {
description: "Unable to convert the token into header value".to_string(), description: "Unable to convert the token into header value".to_string(),
} }

View File

@@ -1,7 +1,7 @@
use crate::config::{Config, GiteaConfig}; use crate::config::{Config, GiteaConfig};
use crate::error::GeneralError; use crate::error::GeneralError;
use reqwest::header::{HeaderMap, ACCEPT, AUTHORIZATION}; use reqwest::header::{HeaderMap, ACCEPT, AUTHORIZATION};
use reqwest::{Body, ClientBuilder, StatusCode}; use reqwest::{ClientBuilder, StatusCode};
use serde::de::DeserializeOwned; use serde::de::DeserializeOwned;
use serde::Serialize; use serde::Serialize;
use url::Url; use url::Url;
@@ -9,7 +9,7 @@ use url::Url;
#[derive(Debug)] #[derive(Debug)]
pub struct Client { pub struct Client {
token: String, token: String,
base_uri: String, // base_uri: String,
} }
fn is_client_for_url(url: &Url, config: &GiteaConfig) -> bool { fn is_client_for_url(url: &Url, config: &GiteaConfig) -> bool {
@@ -29,6 +29,7 @@ pub(crate) fn has_client_for_url(url: &Url, config: &Config) -> bool {
false false
} }
#[allow(dead_code)]
fn client_for_url(url: &Url, config: &Config) -> Result<Client, GeneralError> { fn client_for_url(url: &Url, config: &Config) -> Result<Client, GeneralError> {
for c in &config.gitea { for c in &config.gitea {
if is_client_for_url(url, c) { if is_client_for_url(url, c) {
@@ -46,10 +47,10 @@ impl Client {
Self::new(&config.token, &config.domain) Self::new(&config.token, &config.domain)
} }
pub fn new(token: &String, domain: &String) -> Self { pub fn new(token: &String, _domain: &String) -> Self {
Client { Client {
token: token.clone(), token: token.clone(),
base_uri: format!("https://{}", domain.clone()), // base_uri: format!("https://{}", domain.clone()),
} }
} }

View File

@@ -1,13 +1,11 @@
use crate::error::GeneralError; use crate::error::GeneralError;
use crate::gitea::client::Client;
use crate::gitea::repository::Repository; use crate::gitea::repository::Repository;
use reqwest::Body;
use serde::{Deserialize, Serialize}; use serde::{Deserialize, Serialize};
use url::Url; use url::Url;
#[derive(Debug, Deserialize)] #[derive(Debug, Deserialize)]
pub struct Issue { pub struct Issue {
pub id: u64, // pub id: u64,
pub number: u64, pub number: u64,
pub title: String, pub title: String,
pub body: String, pub body: String,
@@ -20,16 +18,6 @@ pub struct IssueWriteSetBody {
pub body: String, pub body: String,
} }
pub trait IssueClient {
fn get_issue(owner_or_organisation: &String, repo: &String, number: &u64) -> Option<Issue>;
}
impl IssueClient for Client {
fn get_issue(_owner_or_organisation: &String, _repo: &String, number: &u64) -> Option<Issue> {
todo!()
}
}
pub fn issue_html_url_to_api(url: &Url) -> Result<Url, GeneralError> { pub fn issue_html_url_to_api(url: &Url) -> Result<Url, GeneralError> {
let mut parts = url.path_segments().unwrap(); let mut parts = url.path_segments().unwrap();
let domain = parts.next().unwrap(); let domain = parts.next().unwrap();

View File

@@ -2,8 +2,8 @@ use serde::Deserialize;
#[derive(Debug, Deserialize)] #[derive(Debug, Deserialize)]
pub struct Repository { pub struct Repository {
id: u64, // id: u64,
name: String, // name: String,
owner: String, // owner: String,
pub full_name: String, pub full_name: String,
} }

View File

@@ -3,10 +3,6 @@ use crate::error::GeneralError;
use gitlab::{AsyncGitlab, GitlabBuilder}; use gitlab::{AsyncGitlab, GitlabBuilder};
use url::Url; use url::Url;
pub struct ClientProvider {}
impl ClientProvider {}
fn is_client_for_url(url: &Url, config: &GitlabConfig) -> bool { fn is_client_for_url(url: &Url, config: &GitlabConfig) -> bool {
if url.domain() == Some(config.domain.as_str()) { if url.domain() == Some(config.domain.as_str()) {
return true; return true;

View File

@@ -1,13 +1,13 @@
use serde::Deserialize; use serde::Deserialize;
#[derive(Deserialize, Debug)] // #[derive(Deserialize, Debug)]
pub struct HalEntity { // pub struct HalEntity {
#[serde(rename = "_type")] // // #[serde(rename = "_type")]
pub d_type: String, // // pub d_type: String,
} // }
#[derive(Deserialize, Debug, Clone)] #[derive(Deserialize, Debug, Clone)]
pub struct Link { pub struct Link {
pub href: String, pub href: String,
pub title: Option<String>, // pub title: Option<String>,
} }

View File

@@ -1,11 +1,11 @@
use crate::openproject::client::{handle_response_status, Client, OpenProjectError}; use crate::openproject::client::{handle_response_status, Client, OpenProjectError};
use crate::openproject::hal::HalEntity; // use crate::openproject::hal::HalEntity;
use serde::Deserialize; use serde::Deserialize;
#[derive(Deserialize, Debug)] #[derive(Deserialize, Debug)]
pub struct User { pub struct User {
pub href: String, pub href: String,
pub title: String, // pub title: String,
} }
#[derive(Deserialize, Debug)] #[derive(Deserialize, Debug)]
pub struct Links { pub struct Links {
@@ -13,12 +13,12 @@ pub struct Links {
} }
#[derive(Deserialize, Debug)] #[derive(Deserialize, Debug)]
pub struct Root { pub struct Root {
#[serde(rename = "instanceName")] // #[serde(rename = "instanceName")]
pub instance_name: String, // pub instance_name: String,
#[serde(rename = "_links")] #[serde(rename = "_links")]
pub links: Links, pub links: Links,
#[serde(flatten)] // #[serde(flatten)]
pub hal_entity: HalEntity, // pub hal_entity: HalEntity,
} }
pub trait RootClient { pub trait RootClient {

View File

@@ -10,13 +10,10 @@ pub struct UserLink {
} }
#[derive(Deserialize, Debug, Clone)] #[derive(Deserialize, Debug, Clone)]
pub struct User { pub struct User {
#[serde(rename = "_type")] // #[serde(rename = "_type")]
#[allow(unused_variables)] // pub d_type: String,
pub d_type: String, // pub id: u64,
#[allow(unused_variables)] // pub name: String,
pub id: u64,
#[allow(unused_variables)]
pub name: String,
#[serde(rename = "_links")] #[serde(rename = "_links")]
pub d_links: UserLink, pub d_links: UserLink,
} }