feat: Initial setup and logic #1

palexcast · 2023-09-19T06:17:48Z

No description provided.

README.md

package-lock.json

package.json

src/index.ts

boyum · 2023-09-19T06:26:22Z

src/index.ts

+let cachedOctokit: InstanceType<typeof GitHub>;
+
+const run = async () => {
+    const pullRequestTitle = github.context?.payload?.pull_request?.title;


Be sure to document what events this action can be triggered by. I assume pull_request and pull_request_target will provide the same PR title, but if it's triggered by push then we'll have to figure out what PRs are connected to that branch. PR and PR target would be the safest in v1 I think.

src/index.ts

package.json

src/index.ts

Co-authored-by: Sindre Bøyum <[email protected]>

src/index.ts

package.json

boyum · 2023-10-12T09:33:48Z

test/conventional-pr-labels.test.js

+    it('it adds feature label', async () => {
+        const addLabels = sandbox.stub(api, 'addLabels').resolves(undefined)
+        const removeLabel = sandbox.stub(api, 'removeLabel').resolves(undefined)
+        sandbox.stub(process.env, 'GITHUB_EVENT_PATH').value('./test/fixtures/feature.json')
+        await api.main()
+        sandbox.assert.calledWith(removeLabel, 'feature', sandbox.match.any)
+        sandbox.assert.calledWith(removeLabel, 'fix', sandbox.match.any)
+        sandbox.assert.calledWith(removeLabel, 'breaking', sandbox.match.any)
+        sandbox.assert.calledWith(removeLabel, 'ignore-for-release', sandbox.match.any)
+        sandbox.assert.calledWith(addLabels, ['feature'], sandbox.match.any)
+    })


We're testing that the "feature" labels is both removed and added. Shouldn't we test that the "feature" label is NOT tampered with?

Co-authored-by: Sindre Bøyum <[email protected]>

feat: Initial setup and logic

0be7382