[auto] Update compiled version

Ref #25

.github/workflows/build.yml

@@ -3,6 +3,9 @@ on:
   push:
     branches-ignore:
       - dist
+    paths:
+      - src/**
+      - .github/workflows/build.yml
 
 jobs:
   build:
@@ -24,11 +27,10 @@ jobs:
         run: npm run build
 
       - name: Commit changes
-        uses: EndBug/add-and-commit@v2
+        uses: EndBug/add-and-commit@v4.0.0
         with:
-          force: true
           message: "[auto] Update compiled version"
-          path: lib
-          pattern: "*.js"
+          add: lib
+          force: true
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

README.md

@@ -10,8 +10,12 @@ This action lets you choose the path that you want to use when adding & committi
 Add a step like this to your workflow:
 
 ```yaml
-- uses: EndBug/add-and-commit@v2 # You can change this to use a specific version
+- uses: EndBug/add-and-commit@v4 # You can change this to use a specific version
   with:
+    # The arguments for the git add command (see the paragraph below for more info)
+    # Default: '.'
+    add: 'src'
+
     # The name of the user that will be displayed as the author of the commit
     # Default: author of the commit that triggered the run
     author_name: Your Name
@@ -32,15 +36,7 @@ Add a step like this to your workflow:
     # Default: 'Commit from GitHub Actions'
     message: 'Your commit message'
 
-    # The path to stage files from
-    # Default: '.'
-    path: 'src'
-
-    # The pattern that mathces file names
-    # Default: '*.*'
-    pattern: "*.js"
-
-    #  The files to remove
+    #  The arguments for the git rm command (see the paragraph below for more info)
     # Default: ''
     remove: "./dir/old_file.js"
 
@@ -54,9 +50,15 @@ Add a step like this to your workflow:
 The only `env` variable required is the token for the action to run: GitHub generates one automatically, but you need to pass it through `env` to make it available to actions. You can find more about `GITHUB_TOKEN` [here](https://help.github.com/en/articles/virtual-environments-for-github-actions#github_token-secret).  
 With that said, you can just copy the example line and don't worry about it. If you do want to use a different token you can pass that in, but I wouldn't see any possible advantage in doing so.
 
+### Adding files:
+
+The action adds files using a regular `git add` command, so you can put every kind of argument in the `add` option. For example, if you don't want it to use a recursive behavior: `$(find . -maxdepth 1 -name *.js)`.
+The script will not stop if one the git commands fails. E.g.: if your command shows a "fatal: pathspec 'yourFile' did not match any files" error the action will go on.
+
 ### Deleting files:
 
-You can delete files with the `remove` option: that runs a `git remove` command that will stage the files in the given path for removal. Please keep in mind that if the path is wrong the action will stop.
+You can delete files with the `remove` option: that runs a `git rm` command that will stage the files in the given path for removal. 
+The script will not stop if one the git commands fails. E.g.: if your command shows a "fatal: pathspec 'yourFile' did not match any files" error the action will go on.
 
 ### Examples:
 
@@ -86,13 +88,12 @@ jobs:
       run: eslint "src/**" --fix
 
     - name: Commit changes
-      uses: EndBug/add-and-commit@v2 
+      uses: EndBug/add-and-commit@v4
       with:
         author_name: Your Name
         author_email: mail@example.com
         message: "Your commit message"
-        path: "."
-        pattern: "*.js"
+        add: "*.js"
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 ```
@@ -118,11 +119,10 @@ jobs:
       - run: echo "123" > ./pathToRepo/file.txt
 
       # ...and then use the action as you would normally do, but providing the path to the repo
-      - uses: EndBug/add-and-commit@v2
+      - uses: EndBug/add-and-commit@v4
         with:
           message: "Add the very useful text file"
-          path: "."
-          pattern: "*.txt"
+          add: "*.txt"
           cwd: "./pathToRepo/"
           force: true
         env:

action.yml

@@ -2,6 +2,10 @@ name: Add & Commit
 description: Add & commit files from a path directly from GitHub Actions
 
 inputs:
+  add:
+    description: Arguments for the git add command
+    required: false
+    default: "."
   author_name:
     description: The name of the user that will be displayed as the author of the commit
     required: false
@@ -20,22 +24,14 @@ inputs:
     description: The message for the commit
     required: false
     default: Commit from GitHub Actions
-  path:
-    description: The path to stage files from
-    required: false
-    default: "."
-  pattern:
-    description: The pattern that mathces file names
-    required: false
-    default: "*.*"
   remove:
-    description: The files to remove
+    description: Arguments for the git rm command
     required: false
     default: ""
 
 runs:
   using: node12
-  main: lib/main.js
+  main: lib/index.js
 
 branding:
   icon: git-commit

lib/entrypoint.sh

@@ -0,0 +1,73 @@
+#!/bin/bash
+set -u
+
+cd $INPUT_CWD
+echo "Running in $PWD."
+
+# Set up .netrc file with GitHub credentials
+git_setup() {
+    cat <<-EOF >$HOME/.netrc
+        machine github.com
+        login $GITHUB_ACTOR
+        password $GITHUB_TOKEN
+
+        machine api.github.com
+        login $GITHUB_ACTOR
+        password $GITHUB_TOKEN
+EOF
+    chmod 600 $HOME/.netrc
+    git config --global user.email "$INPUT_AUTHOR_EMAIL"
+    git config --global user.name "$INPUT_AUTHOR_NAME"
+}
+
+add() {
+    if $INPUT_FORCE; then f=-f; else f=; fi
+    git add $INPUT_ADD $f
+}
+
+remove() {
+    if [ -n "$INPUT_REMOVE" ]; then git rm $INPUT_REMOVE; fi
+}
+
+# This is needed to make the check work for untracked files
+echo "Staging files..."
+add
+remove
+
+echo "Checking for uncommitted changes in the git working tree..."
+# This section only runs if there have been file changes
+if ! git diff --cached --exit-code; then
+    git_setup
+
+    git fetch
+
+    # Verify if the branch needs to be created
+    if ! git rev-parse --verify --quiet "${GITHUB_REF:11}"; then
+        echo "Creating branch..."
+        git branch "${GITHUB_REF:11}"
+    fi
+
+    # Switch to branch from current workflow run
+    echo "Switching branch..."
+    git checkout "${GITHUB_REF:11}"
+
+    echo "Pulling from remote..."
+    git fetch && git pull
+
+    echo "Resetting files..."
+    git reset
+
+    echo "Adding files..."
+    add
+
+    echo "Removing files..."
+    remove
+
+    echo "Creating commit..."
+    git commit -m "$INPUT_MESSAGE" --author="$INPUT_AUTHOR_NAME <$INPUT_AUTHOR_EMAIL>"
+
+    echo "Pushing to repo..."
+    git push --set-upstream origin "${GITHUB_REF:11}"
+else
+    echo "Working tree clean. Nothing to commit."
+fi

lib/index.js

@@ -0,0 +1 @@
+module.exports=function(e,t){"use strict";var r={};function __webpack_require__(t){if(r[t]){return r[t].exports}var n=r[t]={i:t,l:false,exports:{}};e[t].call(n.exports,n,n.exports,__webpack_require__);n.l=true;return n.exports}__webpack_require__.ab=__dirname+"/";function startup(){return __webpack_require__(131)}return startup()}({87:function(e){e.exports=require("os")},129:function(e){e.exports=require("child_process")},131:function(e,t,r){"use strict";Object.defineProperty(t,"__esModule",{value:true});const n=r(470);const o=r(622);const s=r(129);try{checkInputs();s.execFileSync(r.ab+"entrypoint.sh")}catch(e){console.error(e);n.setFailed(e instanceof Error?e.message:e)}function checkInputs(){var e,t,r;const o=process.env.GITHUB_EVENT_PATH;const s=o&&((t=(e=require(o))===null||e===void 0?void 0:e.head_commit)===null||t===void 0?void 0:t.author);if(s){setDefault("author_name",s.name);setDefault("author_email",s.email)}else{if(!n.getInput("author_name")||!n.getInput("author_email"))n.warning(`Unable to fetch author info: couldn't find ${!o?"event path":!((r=require(o))===null||r===void 0?void 0:r.head_commit)?"commit":"commit author"}.`);setDefault("author_name","Add & Commit Action");setDefault("author_email","actions@github.com")}n.info(`Using '${n.getInput("author_name")} <${n.getInput("author_email")}>' as author.`)}function setDefault(e,t){const r="INPUT_"+e.toUpperCase();if(!process.env[r])process.env[r]=t}},431:function(e,t,r){"use strict";var n=this&&this.__importStar||function(e){if(e&&e.__esModule)return e;var t={};if(e!=null)for(var r in e)if(Object.hasOwnProperty.call(e,r))t[r]=e[r];t["default"]=e;return t};Object.defineProperty(t,"__esModule",{value:true});const o=n(r(87));function issueCommand(e,t,r){const n=new Command(e,t,r);process.stdout.write(n.toString()+o.EOL)}t.issueCommand=issueCommand;function issue(e,t=""){issueCommand(e,{},t)}t.issue=issue;const s="::";class Command{constructor(e,t,r){if(!e){e="missing.command"}this.command=e;this.properties=t;this.message=r}toString(){let e=s+this.command;if(this.properties&&Object.keys(this.properties).length>0){e+=" ";let t=true;for(const r in this.properties){if(this.properties.hasOwnProperty(r)){const n=this.properties[r];if(n){if(t){t=false}else{e+=","}e+=`${r}=${escapeProperty(n)}`}}}}e+=`${s}${escapeData(this.message)}`;return e}}function escapeData(e){return(e||"").replace(/%/g,"%25").replace(/\r/g,"%0D").replace(/\n/g,"%0A")}function escapeProperty(e){return(e||"").replace(/%/g,"%25").replace(/\r/g,"%0D").replace(/\n/g,"%0A").replace(/:/g,"%3A").replace(/,/g,"%2C")}},470:function(e,t,r){"use strict";var n=this&&this.__awaiter||function(e,t,r,n){function adopt(e){return e instanceof r?e:new r(function(t){t(e)})}return new(r||(r=Promise))(function(r,o){function fulfilled(e){try{step(n.next(e))}catch(e){o(e)}}function rejected(e){try{step(n["throw"](e))}catch(e){o(e)}}function step(e){e.done?r(e.value):adopt(e.value).then(fulfilled,rejected)}step((n=n.apply(e,t||[])).next())})};var o=this&&this.__importStar||function(e){if(e&&e.__esModule)return e;var t={};if(e!=null)for(var r in e)if(Object.hasOwnProperty.call(e,r))t[r]=e[r];t["default"]=e;return t};Object.defineProperty(t,"__esModule",{value:true});const s=r(431);const i=o(r(87));const u=o(r(622));var a;(function(e){e[e["Success"]=0]="Success";e[e["Failure"]=1]="Failure"})(a=t.ExitCode||(t.ExitCode={}));function exportVariable(e,t){process.env[e]=t;s.issueCommand("set-env",{name:e},t)}t.exportVariable=exportVariable;function setSecret(e){s.issueCommand("add-mask",{},e)}t.setSecret=setSecret;function addPath(e){s.issueCommand("add-path",{},e);process.env["PATH"]=`${e}${u.delimiter}${process.env["PATH"]}`}t.addPath=addPath;function getInput(e,t){const r=process.env[`INPUT_${e.replace(/ /g,"_").toUpperCase()}`]||"";if(t&&t.required&&!r){throw new Error(`Input required and not supplied: ${e}`)}return r.trim()}t.getInput=getInput;function setOutput(e,t){s.issueCommand("set-output",{name:e},t)}t.setOutput=setOutput;function setFailed(e){process.exitCode=a.Failure;error(e)}t.setFailed=setFailed;function isDebug(){return process.env["RUNNER_DEBUG"]==="1"}t.isDebug=isDebug;function debug(e){s.issueCommand("debug",{},e)}t.debug=debug;function error(e){s.issue("error",e)}t.error=error;function warning(e){s.issue("warning",e)}t.warning=warning;function info(e){process.stdout.write(e+i.EOL)}t.info=info;function startGroup(e){s.issue("group",e)}t.startGroup=startGroup;function endGroup(){s.issue("endgroup")}t.endGroup=endGroup;function group(e,t){return n(this,void 0,void 0,function*(){startGroup(e);let r;try{r=yield t()}finally{endGroup()}return r})}t.group=group;function saveState(e,t){s.issueCommand("save-state",{name:e},t)}t.saveState=saveState;function getState(e){return process.env[`STATE_${e}`]||""}t.getState=getState},622:function(e){e.exports=require("path")}});

lib/main.js

@@ -1 +0,0 @@
-"use strict";var __importStar=this&&this.__importStar||function(a){if(a&&a.__esModule)return a;var b={};if(null!=a)for(var c in a)Object.hasOwnProperty.call(a,c)&&(b[c]=a[c]);return b["default"]=a,b};Object.defineProperty(exports,"__esModule",{value:!0});const core=__importStar(require("@actions/core")),shell=__importStar(require("shelljs")),path=__importStar(require("path"));try{checkInputs(),shell.exec(path.join(__dirname,"../src/entrypoint.sh"))}catch(a){core.setFailed(a)}function checkInputs(){const a=process.env.GITHUB_EVENT_PATH;if(a){const{author:b}=require(a).head_commit;setDefault("author_name",b.name),setDefault("author_email",b.email)}else core.warning("No event path available, unable to fetch author info."),setDefault("author_name","Add & Commit Action"),setDefault("author_email","actions@github.com");core.info(`Using '${core.getInput("author_name")} <${core.getInput("author_email")}>' as author.`)}function setDefault(a,b){const c="INPUT_"+a.toUpperCase();process.env[c]||(process.env[c]=b)}

package.json

@@ -1,13 +1,11 @@
 {
   "name": "add-and-commit",
-  "version": "2.3.0",
+  "version": "4.0.1",
   "description": "Add & commit files from a path directly from GitHub Actions",
-  "main": "lib/main.js",
+  "main": "lib/index.js",
   "scripts": {
-    "build": "npm run compile && npm run minify && rm build/*.js",
-    "compile": "tsc",
-    "minify": "minify build -d lib",
-    "dist": "echo \"Preparing for distribution...\" && npm i --only=prod && git add -f node_modules && git commit -m \"Commit dist files\"",
+    "build": "npm i && ncc build src/main.ts --minify --out lib",
+    "watch": "ncc build src/main.ts --watch --out lib",
     "test": "echo \"Error: no test specified\" && exit 1"
   },
   "repository": {
@@ -28,15 +26,14 @@
   },
   "homepage": "https://github.com/EndBug/add-and-commit#readme",
   "dependencies": {
-    "@actions/core": "^1.1.3",
-    "shelljs": "^0.8.3"
+    "@actions/core": "^1.1.3"
   },
   "devDependencies": {
     "@types/node": "^12.7.12",
     "@types/shelljs": "^0.8.6",
     "@typescript-eslint/eslint-plugin": "^2.12.0",
     "@typescript-eslint/parser": "^2.12.0",
-    "babel-minify": "^0.5.1",
+    "@zeit/ncc": "^0.21.1",
     "eslint": "^6.8.0",
     "typescript": "^3.6.4"
   }

src/entrypoint.sh

@@ -1,5 +1,5 @@
 #!/bin/bash
-set -eu
+set -u
 
 cd $INPUT_CWD
 echo "Running in $PWD."
@@ -22,7 +22,7 @@ EOF
 
 add() {
     if $INPUT_FORCE; then f=-f; else f=; fi
-    find $INPUT_PATH -name "$INPUT_PATTERN" | while read x; do git add $f $x; done
+    git add $INPUT_ADD $f
 }
 
 remove() {

src/main.ts

@@ -1,29 +1,32 @@
-import * as core from '@actions/core'
-import * as shell from 'shelljs'
-import * as path from 'path'
+import { info, setFailed, getInput, warning } from '@actions/core'
+import { join as path } from 'path'
+import { execFileSync } from 'child_process'
 
 try {
   checkInputs()
-  shell.exec(path.join(__dirname, '../src/entrypoint.sh'))
-} catch (e) {
-  core.setFailed(e)
+  execFileSync(path(__dirname, 'entrypoint.sh'))
+} catch (err) {
+  console.error(err)
+  setFailed(err instanceof Error ? err.message : err)
 }
 
 function checkInputs() {
   const eventPath = process.env.GITHUB_EVENT_PATH
-  if (eventPath) {
-    const { author } = require(eventPath).head_commit
+  const author = eventPath && require(eventPath)?.head_commit?.author
+
+  if (author) {
     setDefault('author_name', author.name)
     setDefault('author_email', author.email)
   } else {
-    core.warning('No event path available, unable to fetch author info.')
+    if (!getInput('author_name') || !getInput('author_email')) warning(`Unable to fetch author info: couldn't find ${!eventPath ? 'event path' : !require(eventPath)?.head_commit ? 'commit' : 'commit author'}.`)
     setDefault('author_name', 'Add & Commit Action')
     setDefault('author_email', 'actions@github.com')
   }
-  core.info(`Using '${core.getInput('author_name')} <${core.getInput('author_email')}>' as author.`)
+
+  info(`Using '${getInput('author_name')} <${getInput('author_email')}>' as author.`)
 }
 
 function setDefault(input: string, value: string) {
   const key = 'INPUT_' + input.toUpperCase()
   if (!process.env[key]) process.env[key] = value
-}
+}