Skip to content

updated workflows #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
gignsky wants to merge 5 commits into from
Closed

updated workflows #5

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
653dbcf
updated workflows
gignsky Jul 22, 2025
d298359
removed clippy check
gignsky Jul 22, 2025
3059486
added dependabot
gignsky Jul 22, 2025
3e8e8d3
updated template
gignsky Jul 22, 2025
1bb2351
flake-check baby
gignsky Jul 22, 2025
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
1 change: 1 addition & 0 deletions .github/CODEOWNERS
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* @gignsky
18 changes: 18 additions & 0 deletions .github/dependabot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
version: 2
updates:
# Rust/Cargo dependencies
- package-ecosystem: "cargo"
directory: "/"
schedule:
interval: "weekly"
open-pull-requests-limit: 10
labels:
- "dependencies"
# GitHub Actions dependencies
- package-ecosystem: "github-actions"
directory: "/"
schedule:
interval: "weekly"
open-pull-requests-limit: 5
labels:
- "dependencies"
22 changes: 2 additions & 20 deletions .github/workflows/check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
with:
toolchain: stable
override: true
components: rustfmt, clippy
components: rustfmt

- name: Run cargo test
uses: actions-rs/cargo@v1
Expand All @@ -39,7 +39,7 @@ jobs:
with:
toolchain: stable
override: true
components: rustfmt, clippy
components: rustfmt

- name: Run cargo check
uses: actions-rs/cargo@v1
Expand All @@ -64,21 +64,3 @@ jobs:
with:
command: fmt
args: --all -- --check

clippy:
name: Clippy
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install Rust toolchain
uses: actions-rs/toolchain@v1
with:
toolchain: stable
override: true
components: clippy

- name: Run cargo clippy
uses: actions-rs/cargo@v1
with:
command: clippy
args: --all-targets --all-features -- -D warnings
7 changes: 7 additions & 0 deletions .github/workflows/docs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,13 @@ on:
push:
branches:
- main
pull_request:
branches:
- develop
- release/*
- hotfix/*
- bugfix/*

permissions:
contents: read
pages: write
Expand Down
11 changes: 1 addition & 10 deletions .github/workflows/ci-nix.yml → .github/workflows/flake-check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,5 @@ jobs:
extra_nix_config: |
experimental-features = nix-command flakes
- name: Run nix flake check
run: NIX_SKIP_TEMPLATE=1 nix flake check --show-trace --impure --no-build
run: NIX_SKIP_TEMPLATE=1 nix flake check --show-trace --impure --print-build-logs
Copy link

Copilot AI Jul 22, 2025

Choose a reason for hiding this comment

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

The change from --no-build to --print-build-logs fundamentally alters the behavior - it now performs builds instead of skipping them. This could significantly increase CI execution time and resource usage. Consider if this change is intentional or if you meant to add --print-build-logs alongside --no-build.

Suggested change
run: NIX_SKIP_TEMPLATE=1 nix flake check --show-trace --impure --print-build-logs
run: NIX_SKIP_TEMPLATE=1 nix flake check --show-trace --impure --no-build --print-build-logs

Copilot uses AI. Check for mistakes.

omnix-ci:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: DeterminateSystems/nix-installer-action@main
- name: Install omnix
run: nix --accept-flake-config profile install "github:juspay/omnix"
- name: Run omnix ci
run: om ci --no-build
21 changes: 13 additions & 8 deletions .github/workflows/help-to-readme.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,15 +1,20 @@
name: Update README with --help Output

on:
pull_request:
branches:
- release/*
- hotfix/*
- bugfix/*
- main
push:
branches:
- main
- develop
- release/*
- hotfix/*
- bugfix/*
Copy link

Copilot AI Jul 22, 2025

Choose a reason for hiding this comment

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

There are trailing spaces after the branch pattern. Remove the extra whitespace for consistency.

Suggested change
- bugfix/*
- bugfix/*

Copilot uses AI. Check for mistakes.
pull_request:
branches:
- release/*
- hotfix/*
- bugfix/*
- main

workflow_dispatch:

jobs:
update-readme:
runs-on: ubuntu-latest
Expand Down
41 changes: 0 additions & 41 deletions .github/workflows/template-check.yml
View file
Open in desktop

This file was deleted.

3 changes: 0 additions & 3 deletions nix/modules/template.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@
description = "Include GitHub Actions workflow configuration";
paths = [
".github/**"
"!.github/workflows/template-check.yml"
];
value = true;
}
Expand All @@ -40,7 +39,6 @@
description = "Keep the flake template in the project";
paths = [
"**/template.nix"
".github/workflows/template-check.yml"
];
value = false;
}
Expand All @@ -64,7 +62,6 @@
".github/workflows/help-to-readme.yml" = true;
".github/workflows/release.yml" = true;
"nix/modules/template.nix" = false;
".github/workflows/template-check.yml" = false;
};
packages.default = {
"bin/qux" = true;
Expand Down
Loading