Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 894face

Browse filesBrowse files
Merge pull request #1008 from github/edoardo/no-fail-12.12
Avoid failure if `@types/node` is already 12.12
2 parents 4d339ae + a9095ce commit 894face
Copy full SHA for 894face

File tree

Expand file treeCollapse file tree

1 file changed

+7
-5
lines changed
Filter options
Expand file treeCollapse file tree

1 file changed

+7
-5
lines changed

‎.github/workflows/pr-checks.yml

Copy file name to clipboardExpand all lines: .github/workflows/pr-checks.yml
+7-5Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,13 @@ jobs:
4646
# `npm install` on Linux.
4747
npm install
4848
49-
git config --global user.email "github-actions@github.com"
50-
git config --global user.name "github-actions[bot]"
51-
# The period in `git add --all .` ensures that we stage deleted files too.
52-
git add --all .
53-
git commit -m "Use @types/node=${NODE_TYPES_VERSION}"
49+
if [ ! -z "$(git status --porcelain)" ]; then
50+
git config --global user.email "github-actions@github.com"
51+
git config --global user.name "github-actions[bot]"
52+
# The period in `git add --all .` ensures that we stage deleted files too.
53+
git add --all .
54+
git commit -m "Use @types/node=${NODE_TYPES_VERSION}"
55+
fi
5456
5557
- name: Check generated JS
5658
run: .github/workflows/script/check-js.sh

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.