Skip to content

Commit 0f83016

Browse files
Fix formatting in workflow files: remove extra spaces in branch definitions and improve readability of commands
1 parent 38eee3f commit 0f83016

3 files changed

Lines changed: 19 additions & 11 deletions

File tree

.github/workflows/changelog-check.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ name: Changelog Check
55

66
on:
77
pull_request:
8-
branches: [ main ]
8+
branches: [main]
99
types: [opened, synchronize, reopened, ready_for_review]
1010

1111
jobs:
@@ -29,7 +29,9 @@ jobs:
2929
echo "$CHANGED_FILES"
3030
3131
# Check if any substantive files were changed (not just docs or CI)
32-
SUBSTANTIVE_CHANGES=$(echo "$CHANGED_FILES" | grep -vE '^(\.github/|CHANGELOG\.md|README\.md|LICENSE|\.gitignore)' || true)
32+
SUBSTANTIVE_CHANGES=$(echo "$CHANGED_FILES" | grep -vE \
33+
'^(\.github/|CHANGELOG\.md|README\.md|LICENSE|\.gitignore)' \
34+
|| true)
3335
3436
if [ -z "$SUBSTANTIVE_CHANGES" ]; then
3537
echo "substantive_changes=false" >> $GITHUB_OUTPUT
@@ -57,7 +59,8 @@ jobs:
5759
echo "Checking CHANGELOG.md content..."
5860
5961
# Check if there's content in the Unreleased section
60-
if grep -A 10 "## \[Unreleased\]" CHANGELOG.md | grep -qE "^### (Added|Changed|Deprecated|Removed|Fixed|Security)"; then
62+
if grep -A 10 "## \[Unreleased\]" CHANGELOG.md | grep -qE \
63+
"^### (Added|Changed|Deprecated|Removed|Fixed|Security)"; then
6164
echo "✓ CHANGELOG.md has entries in the Unreleased section"
6265
else
6366
echo "⚠ Warning: CHANGELOG.md may not have entries in the Unreleased section"

.github/workflows/lint-test.yml

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@ name: Lint and Test
55

66
on:
77
pull_request:
8-
branches: [ main ]
8+
branches: [main]
99
push:
10-
branches: [ main ]
10+
branches: [main]
1111
workflow_dispatch: {}
1212

1313
jobs:
@@ -24,20 +24,24 @@ jobs:
2424
uses: actions/setup-python@v6
2525
with:
2626
python-version: '3.11'
27+
continue-on-error: false
2728

2829
- name: Install yamllint
2930
run: |
3031
python -m pip install --upgrade pip
3132
pip install yamllint
33+
continue-on-error: false
3234

3335
- name: Run yamllint
3436
run: |
3537
echo "Running yamllint on workflow files..."
36-
yamllint -d '{extends: default, rules: {line-length: {max: 120}, comments: {min-spaces-from-content: 1}}}' .github/workflows/ || true
37-
continue-on-error: true
38+
yamllint -d \
39+
'{extends: default, rules: {line-length: {max: 120}, comments: {min-spaces-from-content: 1}}}' .github/workflows/ \
40+
|| true
41+
continue-on-error: false
3842

3943
- name: Validate action.yml
4044
run: |
4145
echo "Validating action.yml syntax..."
4246
python -c "import yaml; yaml.safe_load(open('action.yml'))" && echo "✓ action.yml is valid YAML"
43-
47+
continue-on-error: false

.github/workflows/release.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ name: Release and Marketplace
55

66
on:
77
push:
8-
branches: [ main ]
8+
branches: [main]
99
workflow_dispatch:
1010
inputs:
1111
version:
@@ -33,7 +33,8 @@ jobs:
3333
run: |
3434
if grep -q "## \[Unreleased\]" CHANGELOG.md; then
3535
# Check if there's actual content under Unreleased
36-
if grep -A 20 "## \[Unreleased\]" CHANGELOG.md | grep -qE "^### (Added|Changed|Deprecated|Removed|Fixed|Security)"; then
36+
if grep -A 20 "## \[Unreleased\]" CHANGELOG.md /| \
37+
grep -qE "^### (Added|Changed|Deprecated|Removed|Fixed|Security)"; then
3738
echo "has_unreleased=true" >> $GITHUB_OUTPUT
3839
echo "✓ Found unreleased changes in CHANGELOG.md"
3940
else
@@ -51,7 +52,7 @@ jobs:
5152
run: |
5253
# Extract content between [Unreleased] and the next version heading
5354
NOTES=$(awk '/## \[Unreleased\]/,/^## \[/' CHANGELOG.md | sed '1d;$d' | sed '/^$/d')
54-
55+
5556
# Save to output (handle multiline)
5657
echo "release_notes<<EOF" >> $GITHUB_OUTPUT
5758
echo "$NOTES" >> $GITHUB_OUTPUT

0 commit comments

Comments
 (0)