Create 1.3.0 release

2019-12-12 19:10:13 -05:00
19 changed files with 162 additions and 336 deletions
--- a/.gitignore
+++ b/.gitignore
@@ -1,2 +1,2 @@
-# node_modules/
+#node_modules/
 __tests__/runner/*
--- a/README.md
+++ b/README.md
@@ -12,7 +12,6 @@ This action will create a github release and optionally upload an artifact to it
 - **commit**: An optional commit reference. This will be used to create the tag if it does not exist.
 - **draft**: Optionally marks this release as a draft release. Set to `true` to enable.
 - **name**: An optional name for the release. If this is omitted the tag will be used.
- **prerelease**: Optionally marks this release as prerelease. Set to true to enable.
 - **tag**: An optional tag for the release. If this is omitted the git ref will be used (if it is a tag).
 - **token**: (**Required**) The Github token. Typically this will be `${{ secrets.GITHUB_TOKEN }}`.

--- a/tests/Action.test.ts
+++ b/tests/Action.test.ts
@@ -6,7 +6,6 @@ import { ArtifactUploader } from "../src/ArtifactUploader";

 const createMock = jest.fn()
 const getMock = jest.fn()
-const listMock = jest.fn()
 const updateMock = jest.fn()
 const uploadMock = jest.fn()

@@ -20,7 +19,6 @@ const commit = 'commit'
 const draft = true
 const id = 100
 const name = 'name'
-const prerelease = true
 const tag = 'tag'
 const token = 'token'
 const url = 'http://api.example.com'
@@ -29,7 +27,6 @@ describe("Action", () => {
    beforeEach(() => {
        createMock.mockClear()
        getMock.mockClear()
-        listMock.mockClear()
        updateMock.mockClear()
        uploadMock.mockClear()
    })
@@ -39,44 +36,16 @@ describe("Action", () => {

        await action.perform()

-        expect(createMock).toBeCalledWith(tag, body, commit, draft, name, prerelease)
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).not.toBeCalled()
    })

-    it('creates release if no release exists to update', async () => {
-        const action = createAction(true, true)
-        const error = { status: 404 }
-        getMock.mockRejectedValue(error)
-
-        await action.perform()
-
-        expect(createMock).toBeCalledWith(tag, body, commit, draft, name, prerelease)
-        expect(uploadMock).toBeCalledWith(artifacts, url)
-    })
-
-    it('creates release if no draft releases', async () => {
-        const action = createAction(true, true)
-        const error = { status: 404 }
-        getMock.mockRejectedValue(error)
-        listMock.mockResolvedValue({
-            data: [
-                { id: id, draft: false, tag_name: tag }
-            ]
-        })
-
-        await action.perform()
-
-        expect(createMock).toBeCalledWith(tag, body, commit, draft, name, prerelease)
-        expect(uploadMock).toBeCalledWith(artifacts, url)
-
-    })
-
    it('creates release then uploads artifact', async () => {
        const action = createAction(false, true)

        await action.perform()

-        expect(createMock).toBeCalledWith(tag, body, commit, draft, name, prerelease)
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).toBeCalledWith(artifacts, url)
    })

@@ -91,12 +60,12 @@ describe("Action", () => {
            expect(error).toEqual("error")
        }

-        expect(createMock).toBeCalledWith(tag, body, commit, draft, name, prerelease)
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).not.toBeCalled()
    })

    it('throws error when get fails', async () => {
-        const action = createAction(true, true)
+        const action = createAction(true, false)
        const error = {
            errors: [
                {
@@ -104,7 +73,7 @@ describe("Action", () => {
                }
            ]
        }
-
+        
        createMock.mockRejectedValue(error)
        getMock.mockRejectedValue("error")
        expect.hasAssertions()
@@ -113,27 +82,34 @@ describe("Action", () => {
        } catch (error) {
            expect(error).toEqual("error")
        }
-
-        expect(getMock).toBeCalledWith(tag)
-        expect(updateMock).not.toBeCalled()
+        
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).not.toBeCalled()
-
+        
    })

    it('throws error when update fails', async () => {
-        const action = createAction(true, true)
-
+        const action = createAction(true, false)
+        const error = {
+            errors: [
+                {
+                    code: 'already_exists'
+                }
+            ]
+        }
+        
+        createMock.mockRejectedValue(error)
        updateMock.mockRejectedValue("error")
-
        expect.hasAssertions()
        try {
            await action.perform()
        } catch (error) {
            expect(error).toEqual("error")
        }
-
-        expect(updateMock).toBeCalledWith(id, tag, body, commit, draft, name, prerelease)
+        
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).not.toBeCalled()
+        
    })

    it('throws error when upload fails', async () => {
@@ -147,46 +123,46 @@ describe("Action", () => {
            expect(error).toEqual("error")
        }

-        expect(createMock).toBeCalledWith(tag, body, commit, draft, name, prerelease)
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).toBeCalledWith(artifacts, url)
    })

-    it('updates draft release', async () => {
-        const action = createAction(true, true)
-        const error = { status: 404 }
-        getMock.mockRejectedValue(error)
-        listMock.mockResolvedValue({
-            data: [
-                { id: 123, draft: false, tag_name: tag },
-                { id: id, draft: true, tag_name: tag }
-            ]
-        })
-
-        await action.perform()
-
-        expect(updateMock).toBeCalledWith(id, tag, body, commit, draft, name, prerelease)
-        expect(uploadMock).toBeCalledWith(artifacts, url)
-
-    })
-
    it('updates release but does not upload if no artifact', async () => {
        const action = createAction(true, false)
+        const error = {
+            errors: [
+                {
+                    code: 'already_exists'
+                }
+            ]
+        }
+        
+        createMock.mockRejectedValue(error)

        await action.perform()
-
-        expect(updateMock).toBeCalledWith(id, tag, body, commit, draft, name, prerelease)
+        
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).not.toBeCalled()
-
+        
    })

    it('updates release then uploads artifact', async () => {
        const action = createAction(true, true)
+        const error = {
+            errors: [
+                {
+                    code: 'already_exists'
+                }
+            ]
+        }
+        
+        createMock.mockRejectedValue(error)

        await action.perform()
-
-        expect(updateMock).toBeCalledWith(id, tag, body, commit, draft, name, prerelease)
+        
+        expect(createMock).toBeCalledWith(tag, body, commit, draft, name)
        expect(uploadMock).toBeCalledWith(artifacts, url)
-
+        
    })

    function createAction(allowUpdates: boolean, hasArtifact: boolean): Action {
@@ -200,7 +176,6 @@ describe("Action", () => {
            return {
                create: createMock,
                getByTag: getMock,
-                listReleases: listMock,
                update: updateMock,
                uploadArtifact: uploadMock
            }
@@ -216,9 +191,6 @@ describe("Action", () => {
                id: id
            }
        })
-        listMock.mockResolvedValue({
-            data: []
-        })
        updateMock.mockResolvedValue({
            data: {
                upload_url: url
@@ -234,7 +206,6 @@ describe("Action", () => {
                commit: commit,
                draft: draft,
                name: name,
-                prerelease: prerelease,
                tag: tag,
                token: token,
                readArtifact: () => artifactData
--- a/tests/ErrorMessage.test.ts
+++ b/tests/ErrorMessage.test.ts
@@ -14,16 +14,15 @@ describe('ErrorMessage', () => {
                    code: 'already_exists',
                    resource: 'release'
                }
-            ],
-            status: 422
+            ]
        }

-        it('does not have error', () => {
+        it('does not have error', ()=> {
            const errorMessage = new ErrorMessage(error)
            expect(errorMessage.hasErrorWithCode('missing_field')).toBeFalsy()
        })

-        it('has error', () => {
+        it('has error', ()=> {
            const errorMessage = new ErrorMessage(error)
            expect(errorMessage.hasErrorWithCode('missing')).toBeTruthy()
        })
@@ -42,30 +41,22 @@ describe('ErrorMessage', () => {
                    code: 'already_exists',
                    resource: 'release'
                }
-            ],
-            status: 422
+            ]
        }

        const errorMessage = new ErrorMessage(error)

-        const expectedString = "Error 422: something bad happened\nErrors:\n- release does not exist.\n- release already exists."
+        const expectedString = "something bad happened\nErrors:\n- release does not exist.\n- release already exists."
        expect(errorMessage.toString()).toBe(expectedString)
    })

    it('generates message without errors', () => {
        const error = {
-            message: 'something bad happened',
-            status: 422
+            message: 'something bad happened'
        }

        const errorMessage = new ErrorMessage(error)

-        expect(errorMessage.toString()).toBe('Error 422: something bad happened')
-    })
-
-    it('provides error status', () => {
-        const error = { status: 404 }
-        const errorMessage = new ErrorMessage(error)
-        expect(errorMessage.status).toBe(404)
+        expect(errorMessage.toString()).toBe('something bad happened')
    })
 })
--- a/tests/Inputs.test.ts
+++ b/tests/Inputs.test.ts
@@ -133,17 +133,6 @@ describe('Inputs', () => {
        })
    })

-    describe('prerelase', () => {
-        it('returns false', () => {
-            expect(inputs.prerelease).toBe(false)
-        })
-
-        it('returns true', () => {
-            mockGetInput.mockReturnValue('true')
-            expect(inputs.prerelease).toBe(true)
-        })
-    })
-
    describe('tag', () => {
        it('returns input tag', () => {
            mockGetInput.mockReturnValue('tag')
--- a/action.yml
+++ b/action.yml
@@ -29,9 +29,6 @@ inputs:
  name:
    description: 'An optional name for the release. If this is omitted the tag will be used.'
    default: ''
-  prerelease:
-    description: "Optionally marks this release as prerelease. Set to true to enable."
-    default: ''  
  tag:
    description: 'An optional tag for the release. If this is omitted the git ref will be used (if it is a tag).'
    default: ''
--- a/lib/Action.js
+++ b/lib/Action.js
@@ -27,59 +27,34 @@ class Action {
    }
    createOrUpdateRelease() {
        return __awaiter(this, void 0, void 0, function* () {
-            if (this.inputs.allowUpdates) {
-                try {
-                    const getResponse = yield this.releases.getByTag(this.inputs.tag);
-                    return yield this.updateRelease(getResponse.data.id);
-                }
-                catch (error) {
-                    if (this.noPublishedRelease(error)) {
-                        return yield this.updateDraftOrCreateRelease();
-                    }
-                    else {
-                        throw error;
-                    }
-                }
-            }
-            else {
+            try {
                return yield this.createRelease();
            }
-        });
-    }
-    updateRelease(id) {
-        return __awaiter(this, void 0, void 0, function* () {
-            const response = yield this.releases.update(id, this.inputs.tag, this.inputs.body, this.inputs.commit, this.inputs.draft, this.inputs.name, this.inputs.prerelease);
-            return response.data.upload_url;
-        });
-    }
-    noPublishedRelease(error) {
-        const errorMessage = new ErrorMessage_1.ErrorMessage(error);
-        return errorMessage.status == 404;
-    }
-    updateDraftOrCreateRelease() {
-        return __awaiter(this, void 0, void 0, function* () {
-            const draftReleaseId = yield this.findMatchingDraftReleaseId();
-            if (draftReleaseId) {
-                return yield this.updateRelease(draftReleaseId);
+            catch (error) {
+                if (this.releaseAlreadyExisted(error) && this.inputs.allowUpdates) {
+                    return this.updateRelease();
+                }
+                else {
+                    throw error;
+                }
            }
-            else {
-                return yield this.createRelease();
-            }
-        });
-    }
-    findMatchingDraftReleaseId() {
-        var _a;
-        return __awaiter(this, void 0, void 0, function* () {
-            const tag = this.inputs.tag;
-            const response = yield this.releases.listReleases();
-            const releases = response.data;
-            const draftRelease = releases.find(release => release.draft && release.tag_name == tag);
-            return (_a = draftRelease) === null || _a === void 0 ? void 0 : _a.id;
        });
    }
    createRelease() {
        return __awaiter(this, void 0, void 0, function* () {
-            const response = yield this.releases.create(this.inputs.tag, this.inputs.body, this.inputs.commit, this.inputs.draft, this.inputs.name, this.inputs.prerelease);
+            const response = yield this.releases.create(this.inputs.tag, this.inputs.body, this.inputs.commit, this.inputs.draft, this.inputs.name);
+            return response.data.upload_url;
+        });
+    }
+    releaseAlreadyExisted(error) {
+        const errorMessage = new ErrorMessage_1.ErrorMessage(error);
+        return errorMessage.hasErrorWithCode('already_exists');
+    }
+    updateRelease() {
+        return __awaiter(this, void 0, void 0, function* () {
+            const getResponse = yield this.releases.getByTag(this.inputs.tag);
+            const id = getResponse.data.id;
+            const response = yield this.releases.update(id, this.inputs.tag, this.inputs.body, this.inputs.commit, this.inputs.draft, this.inputs.name);
            return response.data.upload_url;
        });
    }
--- a/lib/ArtifactUploader.js
+++ b/lib/ArtifactUploader.js
@@ -8,15 +8,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
        step((generator = generator.apply(thisArg, _arguments || [])).next());
    });
 };
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (Object.hasOwnProperty.call(mod, k)) result[k] = mod[k];
-    result["default"] = mod;
-    return result;
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-const core = __importStar(require("@actions/core"));
 class GithubArtifactUploader {
    constructor(releases) {
        this.releases = releases;
@@ -24,13 +16,7 @@ class GithubArtifactUploader {
    uploadArtifacts(artifacts, uploadUrl) {
        return __awaiter(this, void 0, void 0, function* () {
            artifacts.forEach((artifact) => __awaiter(this, void 0, void 0, function* () {
-                try {
-                    yield this.releases.uploadArtifact(uploadUrl, artifact.contentLength, artifact.contentType, artifact.readFile(), artifact.name);
-                }
-                catch (error) {
-                    const message = `Failed to upload artifact ${artifact.name}. Does it already exist?`;
-                    core.warning(message);
-                }
+                yield this.releases.uploadArtifact(uploadUrl, artifact.contentLength, artifact.contentType, artifact.readFile(), artifact.name);
            }));
        });
    }
--- a/lib/ErrorMessage.js
+++ b/lib/ErrorMessage.js
@@ -15,21 +15,17 @@ class ErrorMessage {
            return [];
        }
    }
-    get status() {
-        return this.error.status;
-    }
    hasErrorWithCode(code) {
        return this.githubErrors.some((err) => err.code == code);
    }
    toString() {
        const message = this.error.message;
        const errors = this.githubErrors;
-        const status = this.status;
        if (errors.length > 0) {
-            return `Error ${status}: ${message}\nErrors:\n${this.errorBulletedList(errors)}`;
+            return `${message}\nErrors:\n${this.errorBulletedList(errors)}`;
        }
        else {
-            return `Error ${status}: ${message}`;
+            return message;
        }
    }
    errorBulletedList(errors) {
--- a/lib/Inputs.js
+++ b/lib/Inputs.js
@@ -58,10 +58,6 @@ class CoreInputs {
        }
        return this.tag;
    }
-    get prerelease() {
-        const draft = core.getInput('prerelease');
-        return draft == 'true';
-    }
    get tag() {
        const tag = core.getInput('tag');
        if (tag) {
--- a/lib/Releases.js
+++ b/lib/Releases.js
@@ -14,28 +14,19 @@ class GithubReleases {
        this.context = context;
        this.git = git;
    }
-    create(tag, body, commitHash, draft, name, prerelease) {
+    create(tag, body, commitHash, draft, name) {
        return __awaiter(this, void 0, void 0, function* () {
            return this.git.repos.createRelease({
                body: body,
                name: name,
                draft: draft,
                owner: this.context.repo.owner,
-                prerelease: prerelease,
                repo: this.context.repo.repo,
                target_commitish: commitHash,
                tag_name: tag
            });
        });
    }
-    listReleases() {
-        return __awaiter(this, void 0, void 0, function* () {
-            return this.git.repos.listReleases({
-                owner: this.context.repo.owner,
-                repo: this.context.repo.repo
-            });
-        });
-    }
    getByTag(tag) {
        return __awaiter(this, void 0, void 0, function* () {
            return this.git.repos.getReleaseByTag({
@@ -45,7 +36,7 @@ class GithubReleases {
            });
        });
    }
-    update(id, tag, body, commitHash, draft, name, prerelease) {
+    update(id, tag, body, commitHash, draft, name) {
        return __awaiter(this, void 0, void 0, function* () {
            return this.git.repos.updateRelease({
                release_id: id,
@@ -53,7 +44,6 @@ class GithubReleases {
                name: name,
                draft: draft,
                owner: this.context.repo.owner,
-                prerelease: prerelease,
                repo: this.context.repo.repo,
                target_commitish: commitHash,
                tag_name: tag
--- a/node_modules/.yarn-integrity
+++ b/node_modules/.yarn-integrity
@@ -11,14 +11,14 @@
    "@actions/core@^1.0.0",
    "@actions/github@^1.0.0",
    "@types/glob@^7.1.1",
-    "@types/jest@^24.0.25",
+    "@types/jest@^24.0.13",
    "@types/node@^12.0.4",
    "add@^2.0.6",
    "glob@^7.1.4",
    "global@^4.4.0",
    "jest-circus@^24.7.1",
-    "jest@^24.9.0",
-    "ts-jest@^24.2.0",
+    "jest@^24.8.0",
+    "ts-jest@^24.0.2",
    "typescript@^3.7.3"
  ],
  "lockfileEntries": {
@@ -81,7 +81,8 @@
    "@types/istanbul-lib-coverage@^2.0.0": "https://registry.yarnpkg.com/@types/istanbul-lib-coverage/-/istanbul-lib-coverage-2.0.1.tgz#42995b446db9a48a11a07ec083499a860e9138ff",
    "@types/istanbul-lib-report@*": "https://registry.yarnpkg.com/@types/istanbul-lib-report/-/istanbul-lib-report-1.1.1.tgz#e5471e7fa33c61358dd38426189c037a58433b8c",
    "@types/istanbul-reports@^1.1.1": "https://registry.yarnpkg.com/@types/istanbul-reports/-/istanbul-reports-1.1.1.tgz#7a8cbf6a406f36c8add871625b278eaf0b0d255a",
-    "@types/jest@^24.0.25": "https://registry.yarnpkg.com/@types/jest/-/jest-24.0.25.tgz#2aba377824ce040114aa906ad2cac2c85351360f",
+    "@types/jest-diff@*": "https://registry.yarnpkg.com/@types/jest-diff/-/jest-diff-20.0.1.tgz#35cc15b9c4f30a18ef21852e255fdb02f6d59b89",
+    "@types/jest@^24.0.13": "https://registry.yarnpkg.com/@types/jest/-/jest-24.0.18.tgz#9c7858d450c59e2164a8a9df0905fc5091944498",
    "@types/minimatch@*": "https://registry.yarnpkg.com/@types/minimatch/-/minimatch-3.0.3.tgz#3dca0e3f33b200fc7d1139c0cd96c1268cadfd9d",
    "@types/node@*": "https://registry.yarnpkg.com/@types/node/-/node-12.7.3.tgz#27b3f40addaf2f580459fdb405222685542f907a",
    "@types/node@^12.0.4": "https://registry.yarnpkg.com/@types/node/-/node-12.7.2.tgz#c4e63af5e8823ce9cc3f0b34f7b998c2171f0c44",
@@ -160,7 +161,7 @@
    "color-name@1.1.3": "https://registry.yarnpkg.com/color-name/-/color-name-1.1.3.tgz#a7d0558bd89c42f795dd42328f740831ca53bc25",
    "combined-stream@^1.0.6": "https://registry.yarnpkg.com/combined-stream/-/combined-stream-1.0.8.tgz#c3d45a8b34fd730631a110a8a2520682b31d5a7f",
    "combined-stream@~1.0.6": "https://registry.yarnpkg.com/combined-stream/-/combined-stream-1.0.8.tgz#c3d45a8b34fd730631a110a8a2520682b31d5a7f",
-    "commander@~2.20.3": "https://registry.yarnpkg.com/commander/-/commander-2.20.3.tgz#fd485e84c03eb4881c20722ba48035e8531aeb33",
+    "commander@~2.20.0": "https://registry.yarnpkg.com/commander/-/commander-2.20.0.tgz#d58bb2b5c1ee8f87b0d340027e9e94e222c5a422",
    "component-emitter@^1.2.1": "https://registry.yarnpkg.com/component-emitter/-/component-emitter-1.3.0.tgz#16e4070fba8ae29b679f2215853ee181ab2eabc0",
    "concat-map@0.0.1": "https://registry.yarnpkg.com/concat-map/-/concat-map-0.0.1.tgz#d8a96bd77fd68df7793a73036a3ba0d5405d477b",
    "console-control-strings@^1.0.0": "https://registry.yarnpkg.com/console-control-strings/-/console-control-strings-1.1.0.tgz#3d7cf4464db6446ea644bf4b39507f9851008e8e",
@@ -252,7 +253,7 @@
    "graceful-fs@^4.1.15": "https://registry.yarnpkg.com/graceful-fs/-/graceful-fs-4.2.2.tgz#6f0952605d0140c1cfdb138ed005775b92d67b02",
    "graceful-fs@^4.1.2": "https://registry.yarnpkg.com/graceful-fs/-/graceful-fs-4.2.2.tgz#6f0952605d0140c1cfdb138ed005775b92d67b02",
    "growly@^1.3.0": "https://registry.yarnpkg.com/growly/-/growly-1.3.0.tgz#f10748cbe76af964b7c96c93c6bcc28af120c081",
-    "handlebars@^4.1.2": "https://registry.yarnpkg.com/handlebars/-/handlebars-4.5.3.tgz#5cf75bd8714f7605713511a56be7c349becb0482",
+    "handlebars@^4.1.2": "https://registry.yarnpkg.com/handlebars/-/handlebars-4.1.2.tgz#b6b37c1ced0306b221e094fc7aca3ec23b131b67",
    "har-schema@^2.0.0": "https://registry.yarnpkg.com/har-schema/-/har-schema-2.0.0.tgz#a94c2224ebcac04782a0d9035521f24735b7ec92",
    "har-validator@~5.1.0": "https://registry.yarnpkg.com/har-validator/-/har-validator-5.1.3.tgz#1ef89ebd3e4996557675eed9893110dc350fa080",
    "has-flag@^3.0.0": "https://registry.yarnpkg.com/has-flag/-/has-flag-3.0.0.tgz#b5d454dc2199ae225699f3467e5a07f3b955bafd",
@@ -324,7 +325,6 @@
    "jest-circus@^24.7.1": "https://registry.yarnpkg.com/jest-circus/-/jest-circus-24.9.0.tgz#8a557683636807d537507eac02ba64c95b686485",
    "jest-cli@^24.9.0": "https://registry.yarnpkg.com/jest-cli/-/jest-cli-24.9.0.tgz#ad2de62d07472d419c6abc301fc432b98b10d2af",
    "jest-config@^24.9.0": "https://registry.yarnpkg.com/jest-config/-/jest-config-24.9.0.tgz#fb1bbc60c73a46af03590719efa4825e6e4dd1b5",
-    "jest-diff@^24.3.0": "https://registry.yarnpkg.com/jest-diff/-/jest-diff-24.9.0.tgz#931b7d0d5778a1baf7452cb816e325e3724055da",
    "jest-diff@^24.9.0": "https://registry.yarnpkg.com/jest-diff/-/jest-diff-24.9.0.tgz#931b7d0d5778a1baf7452cb816e325e3724055da",
    "jest-docblock@^24.3.0": "https://registry.yarnpkg.com/jest-docblock/-/jest-docblock-24.9.0.tgz#7970201802ba560e1c4092cc25cbedf5af5a8ce2",
    "jest-each@^24.9.0": "https://registry.yarnpkg.com/jest-each/-/jest-each-24.9.0.tgz#eb2da602e2a610898dbc5f1f6df3ba86b55f8b05",
@@ -351,7 +351,7 @@
    "jest-watcher@^24.9.0": "https://registry.yarnpkg.com/jest-watcher/-/jest-watcher-24.9.0.tgz#4b56e5d1ceff005f5b88e528dc9afc8dd4ed2b3b",
    "jest-worker@^24.6.0": "https://registry.yarnpkg.com/jest-worker/-/jest-worker-24.9.0.tgz#5dbfdb5b2d322e98567898238a9697bcce67b3e5",
    "jest-worker@^24.9.0": "https://registry.yarnpkg.com/jest-worker/-/jest-worker-24.9.0.tgz#5dbfdb5b2d322e98567898238a9697bcce67b3e5",
-    "jest@^24.9.0": "https://registry.yarnpkg.com/jest/-/jest-24.9.0.tgz#987d290c05a08b52c56188c1002e368edb007171",
+    "jest@^24.8.0": "https://registry.yarnpkg.com/jest/-/jest-24.9.0.tgz#987d290c05a08b52c56188c1002e368edb007171",
    "js-tokens@^3.0.0 || ^4.0.0": "https://registry.yarnpkg.com/js-tokens/-/js-tokens-4.0.0.tgz#19203fb59991df98e3a287050d4647cdeaf32499",
    "js-tokens@^4.0.0": "https://registry.yarnpkg.com/js-tokens/-/js-tokens-4.0.0.tgz#19203fb59991df98e3a287050d4647cdeaf32499",
    "jsbn@~0.1.0": "https://registry.yarnpkg.com/jsbn/-/jsbn-0.1.1.tgz#a5e654c2e5a2deb5f201d96cefbca80c0ef2f513",
@@ -378,7 +378,6 @@
    "load-json-file@^4.0.0": "https://registry.yarnpkg.com/load-json-file/-/load-json-file-4.0.0.tgz#2f5f45ab91e33216234fd53adab668eb4ec0993b",
    "locate-path@^3.0.0": "https://registry.yarnpkg.com/locate-path/-/locate-path-3.0.0.tgz#dbec3b3ab759758071b58fe59fc41871af21400e",
    "lodash.get@^4.4.2": "https://registry.yarnpkg.com/lodash.get/-/lodash.get-4.4.2.tgz#2d177f652fa31e939b4438d5341499dfa3825e99",
-    "lodash.memoize@4.x": "https://registry.yarnpkg.com/lodash.memoize/-/lodash.memoize-4.1.2.tgz#bcc6c49a42a2840ed997f323eada5ecd182e0bfe",
    "lodash.set@^4.3.2": "https://registry.yarnpkg.com/lodash.set/-/lodash.set-4.3.2.tgz#d8757b1da807dde24816b0d6a84bea1a76230b23",
    "lodash.sortby@^4.7.0": "https://registry.yarnpkg.com/lodash.sortby/-/lodash.sortby-4.7.0.tgz#edd14c824e2cc9c1e0b0a1b42bb5210516a42438",
    "lodash.uniq@^4.5.0": "https://registry.yarnpkg.com/lodash.uniq/-/lodash.uniq-4.5.0.tgz#d0225373aeb652adc1bc82e4945339a842754773",
@@ -594,13 +593,13 @@
    "tough-cookie@~2.4.3": "https://registry.yarnpkg.com/tough-cookie/-/tough-cookie-2.4.3.tgz#53f36da3f47783b0925afa06ff9f3b165280f781",
    "tr46@^1.0.1": "https://registry.yarnpkg.com/tr46/-/tr46-1.0.1.tgz#a8b13fd6bfd2489519674ccde55ba3693b706d09",
    "trim-right@^1.0.1": "https://registry.yarnpkg.com/trim-right/-/trim-right-1.0.1.tgz#cb2e1203067e0c8de1f614094b9fe45704ea6003",
-    "ts-jest@^24.2.0": "https://registry.yarnpkg.com/ts-jest/-/ts-jest-24.2.0.tgz#7abca28c2b4b0a1fdd715cd667d65d047ea4e768",
+    "ts-jest@^24.0.2": "https://registry.yarnpkg.com/ts-jest/-/ts-jest-24.0.2.tgz#8dde6cece97c31c03e80e474c749753ffd27194d",
    "tunnel-agent@^0.6.0": "https://registry.yarnpkg.com/tunnel-agent/-/tunnel-agent-0.6.0.tgz#27a5dea06b36b04a0a9966774b290868f0fc40fd",
    "tweetnacl@^0.14.3": "https://registry.yarnpkg.com/tweetnacl/-/tweetnacl-0.14.5.tgz#5ae68177f192d4456269d108afa93ff8743f4f64",
    "tweetnacl@~0.14.0": "https://registry.yarnpkg.com/tweetnacl/-/tweetnacl-0.14.5.tgz#5ae68177f192d4456269d108afa93ff8743f4f64",
    "type-check@~0.3.2": "https://registry.yarnpkg.com/type-check/-/type-check-0.3.2.tgz#5884cab512cf1d355e3fb784f30804b2b520db72",
    "typescript@^3.7.3": "https://registry.yarnpkg.com/typescript/-/typescript-3.7.3.tgz#b36840668a16458a7025b9eabfad11b66ab85c69",
-    "uglify-js@^3.1.4": "https://registry.yarnpkg.com/uglify-js/-/uglify-js-3.7.3.tgz#f918fce9182f466d5140f24bb0ff35c2d32dcc6a",
+    "uglify-js@^3.1.4": "https://registry.yarnpkg.com/uglify-js/-/uglify-js-3.6.0.tgz#704681345c53a8b2079fb6cec294b05ead242ff5",
    "union-value@^1.0.0": "https://registry.yarnpkg.com/union-value/-/union-value-1.0.1.tgz#0b6fe7b835aecda61c6ea4d4f02c14221e109847",
    "universal-user-agent@^2.0.3": "https://registry.yarnpkg.com/universal-user-agent/-/universal-user-agent-2.1.0.tgz#5abfbcc036a1ba490cb941f8fd68c46d3669e8e4",
    "universal-user-agent@^3.0.0": "https://registry.yarnpkg.com/universal-user-agent/-/universal-user-agent-3.0.0.tgz#4cc88d68097bffd7ac42e3b7c903e7481424b4b9",
@@ -644,13 +643,5 @@
    "yargs@^13.3.0": "https://registry.yarnpkg.com/yargs/-/yargs-13.3.0.tgz#4c657a55e07e5f2cf947f8a366567c04a0dedc83"
  },
  "files": [],
-  "artifacts": {
-    "fsevents@1.2.9": [
-      "lib",
-      "lib/binding",
-      "lib/binding/Release",
-      "lib/binding/Release/node-v72-darwin-x64",
-      "lib/binding/Release/node-v72-darwin-x64/fse.node"
-    ]
-  }
+  "artifacts": {}
 }
--- a/package.json
+++ b/package.json
@@ -32,10 +32,10 @@
    "global": "^4.4.0"
  },
  "devDependencies": {
-    "@types/jest": "^24.0.25",
-    "jest": "^24.9.0",
+    "@types/jest": "^24.0.13",
+    "jest": "^24.8.0",
    "jest-circus": "^24.7.1",
-    "ts-jest": "^24.2.0",
+    "ts-jest": "^24.0.2",
    "typescript": "^3.7.3"
  }
 }
--- a/src/Action.ts
+++ b/src/Action.ts
@@ -24,67 +24,45 @@ export class Action {
    }

    private async createOrUpdateRelease(): Promise<string> {
-        if (this.inputs.allowUpdates) {
-            try {
-                const getResponse = await this.releases.getByTag(this.inputs.tag)
-                return await this.updateRelease(getResponse.data.id)
-            } catch (error) {
-                if (this.noPublishedRelease(error)) {
-                    return await this.updateDraftOrCreateRelease()
-                } else {
-                    throw error
-                }
+        try {
+            return await this.createRelease()
+        } catch (error) {
+            if (this.releaseAlreadyExisted(error) && this.inputs.allowUpdates) {
+                return this.updateRelease()
+            } else {
+                throw error
            }
-        } else {
-            return await this.createRelease()
        }
    }

-    private async updateRelease(id: number): Promise<string> {
-        const response = await this.releases.update(
-            id,
-            this.inputs.tag,
-            this.inputs.body,
-            this.inputs.commit,
-            this.inputs.draft,
-            this.inputs.name,
-            this.inputs.prerelease
-        )
-
-        return response.data.upload_url
-    }
-
-    private noPublishedRelease(error: any): boolean {
-        const errorMessage = new ErrorMessage(error)
-        return errorMessage.status == 404
-    }
-
-    private async updateDraftOrCreateRelease(): Promise<string> {
-        const draftReleaseId = await this.findMatchingDraftReleaseId()
-        if (draftReleaseId) {
-            return await this.updateRelease(draftReleaseId)
-        } else {
-            return await this.createRelease()
-        }
-    }
-
-    private async findMatchingDraftReleaseId(): Promise<number | undefined> {
-        const tag = this.inputs.tag
-        const response = await this.releases.listReleases()
-        const releases = response.data
-        const draftRelease = releases.find(release => release.draft && release.tag_name == tag)
-
-        return draftRelease?.id
-    }
-
    private async createRelease(): Promise<string> {
        const response = await this.releases.create(
            this.inputs.tag,
            this.inputs.body,
            this.inputs.commit,
            this.inputs.draft,
-            this.inputs.name,
-            this.inputs.prerelease
+            this.inputs.name
+        )
+
+        return response.data.upload_url
+    }
+
+    private releaseAlreadyExisted(error: any): boolean {
+        const errorMessage = new ErrorMessage(error)
+        return errorMessage.hasErrorWithCode('already_exists')
+    }
+
+    private async updateRelease(): Promise<string> {
+        const getResponse = await this.releases.getByTag(this.inputs.tag)
+        const id = getResponse.data.id
+
+        const response = await this.releases.update(
+            id,
+            this.inputs.tag,
+            this.inputs.body,
+            this.inputs.commit,
+            this.inputs.draft,
+            this.inputs.name
        )

        return response.data.upload_url
--- a/src/ArtifactUploader.ts
+++ b/src/ArtifactUploader.ts
@@ -1,4 +1,3 @@
-import * as core from '@actions/core';
 import { Artifact } from "./Artifact";
 import { Releases } from "./Releases";

@@ -15,16 +14,11 @@ export class GithubArtifactUploader implements ArtifactUploader {

    async uploadArtifacts(artifacts: Artifact[], uploadUrl: string) {
        artifacts.forEach(async artifact => {
-            try {
-                await this.releases.uploadArtifact(uploadUrl,
-                    artifact.contentLength,
-                    artifact.contentType,
-                    artifact.readFile(),
-                    artifact.name)
-            } catch(error) {
-                const message = `Failed to upload artifact ${artifact.name}. Does it already exist?`
-                core.warning(message)
-            }
+            await this.releases.uploadArtifact(uploadUrl,
+                artifact.contentLength,
+                artifact.contentType,
+                artifact.readFile(),
+                artifact.name)
        });
    }
 }
--- a/src/ErrorMessage.ts
+++ b/src/ErrorMessage.ts
@@ -18,10 +18,6 @@ export class ErrorMessage {
        }
    }

-    get status():number {
-        return this.error.status
-    }
-
    hasErrorWithCode(code: String): boolean {
        return this.githubErrors.some((err) => err.code == code)
    }
@@ -29,11 +25,10 @@ export class ErrorMessage {
    toString(): string {
        const message = this.error.message
        const errors = this.githubErrors
-        const status = this.status
        if (errors.length > 0) {
-            return `Error ${status}: ${message}\nErrors:\n${this.errorBulletedList(errors)}`
+            return `${message}\nErrors:\n${this.errorBulletedList(errors)}`
        } else {
-            return `Error ${status}: ${message}`
+            return message
        }
    }

--- a/src/Inputs.ts
+++ b/src/Inputs.ts
@@ -11,7 +11,6 @@ export interface Inputs {
    readonly commit: string
    readonly draft: boolean
    readonly name: string
-    readonly prerelease: boolean
    readonly tag: string
    readonly token: string
 }
@@ -78,11 +77,6 @@ export class CoreInputs implements Inputs {
        return this.tag
    }

-    get prerelease(): boolean {
-        const draft = core.getInput('prerelease')
-        return draft == 'true'
-    }
-
    get tag(): string {
        const tag = core.getInput('tag')
        if (tag) {
--- a/src/Releases.ts
+++ b/src/Releases.ts
@@ -1,6 +1,6 @@
 import { Context } from "@actions/github/lib/context";
 import { GitHub } from "@actions/github";
-import { AnyResponse, Response, ReposCreateReleaseResponse, ReposGetReleaseByTagResponse, ReposListReleasesResponse } from "@octokit/rest";
+import { AnyResponse, Response, ReposCreateReleaseResponse, ReposGetReleaseByTagResponse } from "@octokit/rest";

 export interface Releases {
    create(
@@ -8,22 +8,18 @@ export interface Releases {
        body?: string,
        commitHash?: string,
        draft?: boolean,
-        name?: string,
-        prerelease?: boolean
+        name?: string
    ): Promise<Response<ReposCreateReleaseResponse>>

    getByTag(tag: string): Promise<Response<ReposGetReleaseByTagResponse>>

-    listReleases(): Promise<Response<ReposListReleasesResponse>>
-
    update(
        id: number,
        tag: string,
        body?: string,
        commitHash?: string,
        draft?: boolean,
-        name?: string,
-        prerelease?: boolean
+        name?: string
    ): Promise<Response<ReposCreateReleaseResponse>>

    uploadArtifact(
@@ -49,28 +45,19 @@ export class GithubReleases implements Releases {
        body?: string,
        commitHash?: string,
        draft?: boolean,
-        name?: string,
-        prerelease?: boolean
+        name?: string
    ): Promise<Response<ReposCreateReleaseResponse>> {
        return this.git.repos.createRelease({
            body: body,
            name: name,
            draft: draft,
            owner: this.context.repo.owner,
-            prerelease: prerelease,
            repo: this.context.repo.repo,
            target_commitish: commitHash,
            tag_name: tag
        })
    }

-    async listReleases(): Promise<Response<ReposListReleasesResponse>> {
-        return this.git.repos.listReleases({
-            owner: this.context.repo.owner,
-            repo: this.context.repo.repo
-        })
-    }
-
    async getByTag(tag: string): Promise<Response<ReposGetReleaseByTagResponse>> {
        return this.git.repos.getReleaseByTag({
            owner: this.context.repo.owner,
@@ -85,8 +72,7 @@ export class GithubReleases implements Releases {
        body?: string,
        commitHash?: string,
        draft?: boolean,
-        name?: string,
-        prerelease?: boolean
+        name?: string
    ): Promise<Response<ReposCreateReleaseResponse>> {
        return this.git.repos.updateRelease({
            release_id: id,
@@ -94,7 +80,6 @@ export class GithubReleases implements Releases {
            name: name,
            draft: draft,
            owner: this.context.repo.owner,
-            prerelease: prerelease,
            repo: this.context.repo.repo,
            target_commitish: commitHash,
            tag_name: tag
--- a/yarn.lock
+++ b/yarn.lock
@@ -425,12 +425,17 @@
    "@types/istanbul-lib-coverage" "*"
    "@types/istanbul-lib-report" "*"

-"@types/jest@^24.0.25":
-  version "24.0.25"
-  resolved "https://registry.yarnpkg.com/@types/jest/-/jest-24.0.25.tgz#2aba377824ce040114aa906ad2cac2c85351360f"
-  integrity sha512-hnP1WpjN4KbGEK4dLayul6lgtys6FPz0UfxMeMQCv0M+sTnzN3ConfiO72jHgLxl119guHgI8gLqDOrRLsyp2g==
+"@types/jest-diff@*":
+  version "20.0.1"
+  resolved "https://registry.yarnpkg.com/@types/jest-diff/-/jest-diff-20.0.1.tgz#35cc15b9c4f30a18ef21852e255fdb02f6d59b89"
+  integrity sha512-yALhelO3i0hqZwhjtcr6dYyaLoCHbAMshwtj6cGxTvHZAKXHsYGdff6E8EPw3xLKY0ELUTQ69Q1rQiJENnccMA==
+
+"@types/jest@^24.0.13":
+  version "24.0.18"
+  resolved "https://registry.yarnpkg.com/@types/jest/-/jest-24.0.18.tgz#9c7858d450c59e2164a8a9df0905fc5091944498"
+  integrity sha512-jcDDXdjTcrQzdN06+TSVsPPqxvsZA/5QkYfIZlq1JMw7FdP5AZylbOc+6B/cuDurctRe+MziUMtQ3xQdrbjqyQ==
  dependencies:
-    jest-diff "^24.3.0"
+    "@types/jest-diff" "*"

 "@types/minimatch@*":
  version "3.0.3"
@@ -882,10 +887,10 @@ combined-stream@^1.0.6, combined-stream@~1.0.6:
  dependencies:
    delayed-stream "~1.0.0"

-commander@~2.20.3:
-  version "2.20.3"
-  resolved "https://registry.yarnpkg.com/commander/-/commander-2.20.3.tgz#fd485e84c03eb4881c20722ba48035e8531aeb33"
-  integrity sha512-GpVkmM8vF2vQUkj2LvZmD35JxeJOLCwJ9cUkugyk2nuhbv3+mJvpLYYt+0+USMxE+oj+ey/lJEnhZw75x/OMcQ==
+commander@~2.20.0:
+  version "2.20.0"
+  resolved "https://registry.yarnpkg.com/commander/-/commander-2.20.0.tgz#d58bb2b5c1ee8f87b0d340027e9e94e222c5a422"
+  integrity sha512-7j2y+40w61zy6YC2iRNpUe/NwhNyoXrYpHMrSunaMG64nRnaf96zO/KMQR4OyN/UnE5KLyEBnKHd4aG3rskjpQ==

 component-emitter@^1.2.1:
  version "1.3.0"
@@ -1411,9 +1416,9 @@ growly@^1.3.0:
  integrity sha1-8QdIy+dq+WS3yWyTxrzCivEgwIE=

 handlebars@^4.1.2:
-  version "4.5.3"
-  resolved "https://registry.yarnpkg.com/handlebars/-/handlebars-4.5.3.tgz#5cf75bd8714f7605713511a56be7c349becb0482"
-  integrity sha512-3yPecJoJHK/4c6aZhSvxOyG4vJKDshV36VHp0iVCDVh7o9w2vwi3NSnL2MMPj3YdduqaBcu7cGbggJQM0br9xA==
+  version "4.1.2"
+  resolved "https://registry.yarnpkg.com/handlebars/-/handlebars-4.1.2.tgz#b6b37c1ced0306b221e094fc7aca3ec23b131b67"
+  integrity sha512-nvfrjqvt9xQ8Z/w0ijewdD/vvWDTOweBUm96NTr66Wfvo1mJenBLwcYmPs3TIBP5ruzYGD7Hx/DaM9RmhroGPw==
  dependencies:
    neo-async "^2.6.0"
    optimist "^0.6.1"
@@ -1867,7 +1872,7 @@ jest-config@^24.9.0:
    pretty-format "^24.9.0"
    realpath-native "^1.1.0"

-jest-diff@^24.3.0, jest-diff@^24.9.0:
+jest-diff@^24.9.0:
  version "24.9.0"
  resolved "https://registry.yarnpkg.com/jest-diff/-/jest-diff-24.9.0.tgz#931b7d0d5778a1baf7452cb816e325e3724055da"
  integrity sha512-qMfrTs8AdJE2iqrTp0hzh7kTd2PQWrsFyj9tORoKmu32xjPjeE4NyjVRDz8ybYwqS2ik8N4hsIpiVTyFeo2lBQ==
@@ -2162,7 +2167,7 @@ jest-worker@^24.6.0, jest-worker@^24.9.0:
    merge-stream "^2.0.0"
    supports-color "^6.1.0"

-jest@^24.9.0:
+jest@^24.8.0:
  version "24.9.0"
  resolved "https://registry.yarnpkg.com/jest/-/jest-24.9.0.tgz#987d290c05a08b52c56188c1002e368edb007171"
  integrity sha512-YvkBL1Zm7d2B1+h5fHEOdyjCG+sGMz4f8D86/0HiqJ6MB4MnDc8FgP5vdWsGnemOQro7lnYo8UakZ3+5A0jxGw==
@@ -2324,11 +2329,6 @@ lodash.get@^4.4.2:
  resolved "https://registry.yarnpkg.com/lodash.get/-/lodash.get-4.4.2.tgz#2d177f652fa31e939b4438d5341499dfa3825e99"
  integrity sha1-LRd/ZS+jHpObRDjVNBSZ36OCXpk=

-lodash.memoize@4.x:
-  version "4.1.2"
-  resolved "https://registry.yarnpkg.com/lodash.memoize/-/lodash.memoize-4.1.2.tgz#bcc6c49a42a2840ed997f323eada5ecd182e0bfe"
-  integrity sha1-vMbEmkKihA7Zl/Mj6tpezRguC/4=
-
 lodash.set@^4.3.2:
  version "4.3.2"
  resolved "https://registry.yarnpkg.com/lodash.set/-/lodash.set-4.3.2.tgz#d8757b1da807dde24816b0d6a84bea1a76230b23"
@@ -3525,16 +3525,15 @@ trim-right@^1.0.1:
  resolved "https://registry.yarnpkg.com/trim-right/-/trim-right-1.0.1.tgz#cb2e1203067e0c8de1f614094b9fe45704ea6003"
  integrity sha1-yy4SAwZ+DI3h9hQJS5/kVwTqYAM=

-ts-jest@^24.2.0:
-  version "24.2.0"
-  resolved "https://registry.yarnpkg.com/ts-jest/-/ts-jest-24.2.0.tgz#7abca28c2b4b0a1fdd715cd667d65d047ea4e768"
-  integrity sha512-Yc+HLyldlIC9iIK8xEN7tV960Or56N49MDP7hubCZUeI7EbIOTsas6rXCMB4kQjLACJ7eDOF4xWEO5qumpKsag==
+ts-jest@^24.0.2:
+  version "24.0.2"
+  resolved "https://registry.yarnpkg.com/ts-jest/-/ts-jest-24.0.2.tgz#8dde6cece97c31c03e80e474c749753ffd27194d"
+  integrity sha512-h6ZCZiA1EQgjczxq+uGLXQlNgeg02WWJBbeT8j6nyIBRQdglqbvzDoHahTEIiS6Eor6x8mK6PfZ7brQ9Q6tzHw==
  dependencies:
    bs-logger "0.x"
    buffer-from "1.x"
    fast-json-stable-stringify "2.x"
    json5 "2.x"
-    lodash.memoize "4.x"
    make-error "1.x"
    mkdirp "0.x"
    resolve "1.x"
@@ -3566,11 +3565,11 @@ typescript@^3.7.3:
  integrity sha512-Mcr/Qk7hXqFBXMN7p7Lusj1ktCBydylfQM/FZCk5glCNQJrCUKPkMHdo9R0MTFWsC/4kPFvDS0fDPvukfCkFsw==

 uglify-js@^3.1.4:
-  version "3.7.3"
-  resolved "https://registry.yarnpkg.com/uglify-js/-/uglify-js-3.7.3.tgz#f918fce9182f466d5140f24bb0ff35c2d32dcc6a"
-  integrity sha512-7tINm46/3puUA4hCkKYo4Xdts+JDaVC9ZPRcG8Xw9R4nhO/gZgUM3TENq8IF4Vatk8qCig4MzP/c8G4u2BkVQg==
+  version "3.6.0"
+  resolved "https://registry.yarnpkg.com/uglify-js/-/uglify-js-3.6.0.tgz#704681345c53a8b2079fb6cec294b05ead242ff5"
+  integrity sha512-W+jrUHJr3DXKhrsS7NUVxn3zqMOFn0hL/Ei6v0anCIMoKC93TjcflTagwIHLW7SfMFfiQuktQyFVCFHGUE0+yg==
  dependencies:
-    commander "~2.20.3"
+    commander "~2.20.0"
    source-map "~0.6.1"

 union-value@^1.0.0: