Skip to content
This repository has been archived by the owner on May 5, 2024. It is now read-only.

Commit

Permalink
We have pursuing more efficient git cloning strategies, which led us …
Browse files Browse the repository at this point in the history
…to a code review of how we structured the git command. We noticed that there was an overridable clone parameter gitCloneOptions that we could use. We are defaulting it to the current best practice filter=tree:0, and removed the hard coded lines
  • Loading branch information
Thomas Branch authored and Thomas Branch committed Oct 13, 2022
1 parent 2b631df commit 21e4b4c
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 5 deletions.
2 changes: 1 addition & 1 deletion bin/stampede-worker.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ const conf = require("rc-house")("stampede", {
environmentVariablePrefix: "STAMP_",
shell: "/bin/bash",
gitClone: "https",
gitCloneOptions: "",
gitCloneOptions: "--filter=tree:0",
defaultGitCloneDepth: 15,
gitMerge: false,
stdoutLogFile: "stdout.log",
Expand Down
3 changes: 0 additions & 3 deletions lib/workingDirectory.js
Original file line number Diff line number Diff line change
Expand Up @@ -194,9 +194,6 @@ async function cloneRepo(
"git clone -b " +
branch +
" --single-branch " +
" --depth " +
depth +
" " +
cloneOptions +
" " +
cloneUrl +
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "stampede-worker",
"version": "1.3.0",
"version": "1.3.1",
"description": "A centralized job queue system",
"main": "bin/stampede-worker.js",
"scripts": {
Expand Down

0 comments on commit 21e4b4c

Please sign in to comment.