-
Notifications
You must be signed in to change notification settings - Fork 2
Oblackmon3/rebase polymer develop to match v0.0.25 #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.
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
base: polymer-develop
Are you sure you want to change the base?
Changes from all commits
8c5b8f4
ba9d420
8a1d08f
dc7abc1
7c60f72
ab0f820
a47071d
42dee51
091f964
b6cf66f
b9a3974
93e6514
7fb73f4
a166b10
6758be0
2beb081
1a52389
879f0c4
3cbbe28
ef069f0
1433983
2a0574f
1caff45
c558d14
78dc7a4
14130a8
03921f4
d223cde
a84317d
90d1cc5
3c55d67
22ff31f
8808cb0
473b46c
19d4392
b535bbf
5bc0ed9
183cb73
f4c3b51
47fd410
9c74b3a
b239cb8
1b1531b
e71c985
75edd2a
4b8dee8
1bb3c81
3f5c124
2fc702b
f0c0520
2ec63ad
9a11434
9c352a8
0b422b0
ab19be7
65d3fc5
2918b09
f770b95
8d28b89
6c87a3f
f2b177a
b84cf75
1380dac
18c4aed
ba5810d
4df7840
82a6cfe
0b1555e
73787a0
b387991
21f9a1c
b903700
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,38 @@ | ||
| name: prune | ||
|
|
||
| on: | ||
| workflow_dispatch: | ||
| schedule: | ||
| - cron: "0 0 * * *" | ||
| push: | ||
| branches: | ||
| - main | ||
|
|
||
| jobs: | ||
| clean: | ||
| runs-on: ubuntu-latest | ||
| name: Prune images | ||
| steps: | ||
| - name: Login to GitHub Container Registry | ||
| uses: docker/login-action@v3.0.0 | ||
| with: | ||
| registry: ghcr.io | ||
| username: ${{ github.actor }} | ||
| password: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Fetch multi-platform package version SHAs | ||
| id: multi-arch-digests | ||
| run: | | ||
| erpc=$(docker manifest inspect ghcr.io/erpc/erpc | jq -r '.manifests.[] | .digest' | paste -s -d ' ' -) | ||
| echo "multi-arch-digests=$erpc" >> $GITHUB_OUTPUT | ||
|
|
||
| - uses: snok/container-retention-policy@v3.0.0 | ||
| with: | ||
| account: erpc | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| image-names: "erpc" | ||
| image-tags: "!latest !main !*.*.*" | ||
| skip-shas: ${{ steps.multi-arch-digests.outputs.multi-arch-digests }} | ||
| tag-selection: both | ||
| cut-off: 0ms | ||
| dry-run: true |
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
|
|
@@ -25,4 +25,4 @@ jobs: | |||
| - name: Upload coverage reports to Codecov | ||||
| uses: codecov/codecov-action@v4.0.1 | ||||
| with: | ||||
| token: ${{ secrets.CODECOV_TOKEN }} | ||||
| args: -exclude-dir=test -tests=false ./... | ||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove duplicated Gosec arguments This line appears to be an unintentional duplication of the Gosec Security Scanner arguments. It's redundant and may cause confusion. Please remove this line: - args: -exclude-dir=test -tests=false ./...📝 Committable suggestion
Suggested change
|
||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,4 +10,5 @@ coverage.txt | |
| .DS_Store | ||
| erpc.yaml | ||
| .generated-go-semantic-release-changelog.md | ||
| .semrel/ | ||
| .semrel/ | ||
| __debug* | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| package main | ||
|
|
||
| import ( | ||
| "github.com/rs/zerolog" | ||
| ) | ||
|
|
||
| func init() { | ||
| zerolog.SetGlobalLevel(zerolog.Disabled) | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix: Use double quotes in short SHA generation
The short SHA generation step is a good addition, but there's a potential issue flagged by the static analysis tool.
To fix the shellcheck warning and prevent potential word splitting issues, use double quotes around the command substitution:
🧰 Tools
🪛 actionlint