Change default for creating artifacts on drafts to false

__tests__/main.test.ts

@@ -209,7 +209,7 @@ describe('action', () => {
     )
   })
 
-  it('skips creating a score artifact by default', async () => {
+  it('skips creating a score artifact when that feature is not enabled', async () => {
     // Mock the @actions/github context.
     Object.defineProperty(github, 'context', {
       value: pullRequestEventContext()
@@ -219,7 +219,28 @@ describe('action', () => {
 
     expect(runMock).toHaveReturned()
     expect(setFailedMock).not.toHaveBeenCalled()
-    expect(infoMock).toHaveBeenCalledWith('Skipping score artifact creation')
+    expect(infoMock).toHaveBeenCalledWith(
+      'Skipping score artifact creation because it has not been enabled'
+    )
+  })
+
+  it('skips creating a score artifact on draft pull requests by default', async () => {
+    // Mock the @actions/github context.
+    Object.defineProperty(github, 'context', {
+      value: pullRequestEventContext({ draft: true })
+    })
+
+    loadConfigurationMock.mockImplementation(() => ({
+      archiving: { persistScoreArtifact: true }
+    }))
+
+    await main.run()
+
+    expect(runMock).toHaveReturned()
+    expect(setFailedMock).not.toHaveBeenCalled()
+    expect(infoMock).toHaveBeenCalledWith(
+      'Skipping score artifact creation on a draft pull request'
+    )
   })
 
   it('runs the workflow sucessfully when optIns configuration is present and the pull request author is in it', async () => {

docs/development.md

@@ -14,4 +14,8 @@ The result of that command (changes to the [`dist`](./dist) directory and other
 
 ## Regenerating the Typescript interface for the configuration schema
 
-Follow the same [procedure](https://github.com/lerebear/sizeup-core#regenerating-the-typescript-interface-for-the-configuration-schema) outlined for this process in the `sizeup-core` repository.
+The [configuration JSON schema](../src/config/schema.json) acts as the source of truth for the configuration file format that this workflow accepts: each change to configuration should begin with a change to that schema. Once that change has been made, we should regenerate the corresponding [`Configuration`](../src/configuration.ts) interface in Typescript using the `generate:config` script:
+
+```sh
+npm run generate:config
+```