Merge pull request #25 from JasonEtco/inputs

Use inputs instead of args for custom filename

README.md

@@ -46,13 +46,13 @@ This example will create a new issue with a title like **Weekly Radar Saturday,
 
 ### Custom templates
 
-Don't want to use `.github/ISSUE_TEMPLATE.md`? You can pass an argument pointing the action to a different template:
+Don't want to use `.github/ISSUE_TEMPLATE.md`? You can pass an input pointing the action to a different template:
 
 ```yaml
 steps:
   - uses: JasonEtco/create-an-issue@master
     env: 
       GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
     with:
-      args: .github/some-other-template.md
+      filename: .github/some-other-template.md
 ```

action.yml

@@ -6,3 +6,7 @@ runs:
 branding:
   icon: alert-circle
   color: gray-dark
+inputs:
+  filename:
+    description: The name of the file to use as the issue template
+    default: .github/ISSUE_TEMPLATE.md

index.js

@@ -1,3 +1,4 @@
+const core = require('@actions/core')
 const { Toolkit } = require('actions-toolkit')
 const fm = require('front-matter')
 const nunjucks = require('nunjucks')
@@ -9,7 +10,7 @@ function listToArray (list) {
 }
 
 Toolkit.run(async tools => {
-  const template = tools.arguments._[0] || '.github/ISSUE_TEMPLATE.md'
+  const template = core.getInput('filename') || '.github/ISSUE_TEMPLATE.md'
   const env = nunjucks.configure({ autoescape: false })
   env.addFilter('date', dateFilter)
 

package.json

@@ -9,6 +9,7 @@
   "author": "Jason Etcovitch <[email protected]>",
   "license": "MIT",
   "dependencies": {
+    "@actions/core": "^1.1.0",
     "actions-toolkit": "^2.2.0",
     "front-matter": "^3.0.2",
     "js-yaml": "^3.13.1",

tests/index.test.js

@@ -30,6 +30,9 @@ describe('create-an-issue', () => {
 
     tools.exit.success = jest.fn()
     tools.exit.failure = jest.fn()
+
+    // Ensure that the filename input isn't set at the start of a test
+    delete process.env.INPUT_FILENAME
   })
 
   it('creates a new issue', async () => {
@@ -41,7 +44,7 @@ describe('create-an-issue', () => {
   })
 
   it('creates a new issue from a different template', async () => {
-    tools.arguments._ = ['.github/different-template.md']
+    process.env.INPUT_FILENAME = '.github/different-template.md'
     tools.context.payload = { repository: { owner: { login: 'JasonEtco' }, name: 'waddup' } }
     await actionFn(tools)
     expect(params).toMatchSnapshot()
@@ -50,22 +53,22 @@ describe('create-an-issue', () => {
   })
 
   it('creates a new issue with some template variables', async () => {
-    tools.arguments._[0] = '.github/variables.md'
+    process.env.INPUT_FILENAME = '.github/variables.md'
     await actionFn(tools)
     expect(tools.log.success).toHaveBeenCalled()
     expect(tools.log.success.mock.calls).toMatchSnapshot()
   })
 
   it('creates a new issue with assignees, labels and a milestone', async () => {
-    tools.arguments._[0] = '.github/kitchen-sink.md'
+    process.env.INPUT_FILENAME = '.github/kitchen-sink.md'
     await actionFn(tools)
     expect(params).toMatchSnapshot()
     expect(tools.log.success).toHaveBeenCalled()
     expect(tools.log.success.mock.calls).toMatchSnapshot()
   })
 
   it('creates a new issue with assignees and labels as comma-delimited strings', async () => {
-    tools.arguments._[0] = '.github/split-strings.md'
+    process.env.INPUT_FILENAME = '.github/split-strings.md'
     await actionFn(tools)
     expect(params).toMatchSnapshot()
     expect(tools.log.success).toHaveBeenCalled()