Skip to content

Commit 7147295

Browse files
committed
fix(CI): make workflows more robust, replacing ls with find
1 parent cffcb65 commit 7147295

File tree

2 files changed

+10
-7
lines changed

2 files changed

+10
-7
lines changed

.github/workflows/release.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,14 +72,15 @@ jobs:
7272

7373
- name: Validate changesets
7474
run: |
75-
if [ -z "$(ls .changeset/*.md 2>/dev/null)" ]; then
75+
# Check if .changeset directory exists and contains .md files
76+
if [ ! -d .changeset ] || [ -z "$(find .changeset -name '*.md' -type f 2>/dev/null | head -1)" ]; then
7677
echo "❌ No changesets found. Please create changesets locally with 'pnpm changeset' before releasing."
7778
echo "Changesets should be created during development, not during release."
7879
exit 1
7980
else
80-
echo "✅ Found $(ls .changeset/*.md | wc -l) changeset(s)"
81+
echo "✅ Found $(find .changeset -name '*.md' -type f | wc -l) changeset(s)"
8182
echo "Changesets:"
82-
ls .changeset/*.md
83+
find .changeset -name '*.md' -type f
8384
fi
8485
8586
- name: Create Release Pull Request or Publish
@@ -100,13 +101,13 @@ jobs:
100101
echo "🔍 This is a dry run. The following changes would be made:"
101102
echo ""
102103
echo "📋 Changeset contents:"
103-
for file in .changeset/*.md; do
104+
while IFS= read -r -d '' file; do
104105
if [ -f "$file" ]; then
105106
echo "--- $file ---"
106107
cat "$file"
107108
echo ""
108109
fi
109-
done
110+
done < <(find .changeset -name '*.md' -type f -print0 2>/dev/null)
110111
111112
echo "📦 Version changes that would be applied:"
112113
pnpm changeset version --dry-run

.github/workflows/tests.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ jobs:
4949
# Check if any packages have changed
5050
if [[ "${{ steps.quality-changes.outputs.react }}" == "true" ]] || [[ "${{ steps.quality-changes.outputs.angular }}" == "true" ]]; then
5151
# Check if .changeset directory exists and contains .md files
52-
if [ ! -d .changeset ] || [ -z "$(ls .changeset/*.md 2>/dev/null)" ]; then
52+
if [ ! -d .changeset ] || [ -z "$(find .changeset -name '*.md' -type f 2>/dev/null | head -1)" ]; then
5353
echo "❌ Changes detected in packages but no changesets found."
5454
echo ""
5555
echo "Please create a changeset with 'pnpm changeset' for changes that should be released."
@@ -67,7 +67,7 @@ jobs:
6767
echo "✅ Found changesets for package changes"
6868
echo ""
6969
echo "Changesets:"
70-
ls .changeset/*.md
70+
find .changeset -name '*.md' -type f
7171
fi
7272
else
7373
echo "ℹ️ No package changes detected, skipping changeset check"
@@ -79,6 +79,8 @@ jobs:
7979
steps:
8080
- name: Checkout code
8181
uses: actions/checkout@v4
82+
with:
83+
fetch-depth: 0
8284

8385
- name: Setup Node.js
8486
uses: actions/setup-node@v4

0 commit comments

Comments
 (0)