Skip to content

Commit

Permalink
fix: Inefficient triggering mechanisms in audit.yml and ci.yml, the p…
Browse files Browse the repository at this point in the history
…ermissions field in cd.yml is defined globally (#83)
  • Loading branch information
kurosakishigure authored Dec 16, 2024
1 parent a7553e2 commit 69f7f3b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/audit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@ name: Security audit
on:
schedule:
- cron: '0 0 * * *'
push:
pull_request:
paths:
- '**/Cargo.toml'
- 'Cargo.toml'

jobs:
cargo-deny:
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,10 @@ on:
push:
tags: [ '[0-9]+.[0-9]+.[0-9]+', '[0-9]+.[0-9]+.[0-9]+-canary.[0-9]+' ]

permissions:
contents: write

jobs:
publish:
permissions:
contents: write
name: Publishing for ${{ matrix.job.os }}
runs-on: ${{ matrix.job.os }}
strategy:
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
name: CI

on:
push:
tags: [ '[0-9]+.[0-9]+.[0-9]+', '[0-9]+.[0-9]+.[0-9]+-canary.[0-9]+' ]
pull_request:
branches: [ 'canary' ]

Expand Down

0 comments on commit 69f7f3b

Please sign in to comment.