tools: avoid fetch extra commits when validating commit messages · nodejs/node@c700de3 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 635
• Pull requests 756
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit c700de3

Browse filesBrowse the repository at this point in the historyBrowse files

aduh95

authored and

targos

committed

tools: avoid fetch extra commits when validating commit messages

PR-URL: #39128 Reviewed-By: Mary Marchini <oss@mmarchini.me>

1 parent e9388c8 commit c700de3
Copy full SHA for c700de3

1 file changed

+4-2Lines changed: 4 additions & 2 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• .github/workflows
  • commit-lint.yml

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎.github/workflows/commit-lint.yml‎

Copy file name to clipboardExpand all lines: .github/workflows/commit-lint.yml

+4-2Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -9,10 +9,12 @@ jobs:
9	9	lint-commit-message:
10	10	runs-on: ubuntu-latest
11	11	steps:
	12	+ - name: Compute number of commits in the PR
	13	+ id: nb-of-commits
	14	+ run: echo "::set-output name=nb::$((${{ github.event.pull_request.commits }} + 1))"
12	15	- uses: actions/checkout@v2
13	16	with:
14		- # Last 100 commits should be enough for a PR
15		- fetch-depth: 100
	17	+ fetch-depth: ${{ steps.nb-of-commits.outputs.nb }}
16	18	- name: Install Node.js
17	19	uses: actions/setup-node@v2
18	20	with:

0 commit comments

Comments

0 (0)