Skip to content
This repository has been archived by the owner on Mar 12, 2024. It is now read-only.

Feature/simplified labeling #376

Closed
wants to merge 5 commits into from
Closed

Feature/simplified labeling #376

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
4cbf79f
Fix imports
EstebanDalelR Nov 13, 2023
c99fa83
Simplify label posting
EstebanDalelR Nov 13, 2023
abb007f
Remove issue numbre requirement as its global
EstebanDalelR Nov 15, 2023
53d3e3b
Improve logging
EstebanDalelR Nov 15, 2023
387756a
Async functions
EstebanDalelR Nov 23, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 14 additions & 29 deletions utils/actions/labelPullRequest.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,7 @@
const { Configuration, OpenAIApi } = require("openai");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file contains at least one console log. Please remove any present.

import { App } from "@octokit/app";
import { successPosthogTracking } from "../../utils/api/posthogTracking";
import {
failedToFetchResponse,
missingParamsResponse,
} from "../../utils/api/responses";
import validateParams from "../../utils/api/validateParams";
import { failedToFetchResponse } from "../../utils/api/responses";

const configuration = new Configuration({
apiKey: process.env.OPEN_AI_KEY,
Expand Down Expand Up @@ -67,36 +63,25 @@ export default async function flagPullRequest({
TAKE_A_DEEPER_DIVE: "👀 Take a deeper dive",
DONT_MERGE: "🚨 Don't Merge",
};
function deleteLabel(labelName: string) {
octokit
.request(
"DELETE /repos/{owner}/{repo}/issues/{issue_number}/labels/{name}",
{
owner,
repo,
issue_number,
name: labelName,
}
)
.catch((error) => {
console.error("Label not found", error);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file contains at least one console log. Please remove any present.

});
}
function addLabel(labelName: string) {
octokit
.request(
"POST /repos/{owner}/{repo}/issues/{issue_number}/labels", //add label

async function modifyLabel(issue_number, labelName, action) {
try {
await octokit.request(
`/${action} /repos/{owner}/{repo}/issues/{issue_number}/labels`,
{
owner,
repo,
issue_number,
labels: [labelName],
labels: action === "POST" ? [labelName] : labelName,
}
)
.catch((error) => {
console.error("add laber", error);
});
);
} catch (error) {
console.error(`Error during label ${action}`, error);
}
}
const addLabel = (labelName) => modifyLabel(issue_number, labelName, "POST");
const deleteLabel = (labelName) =>
modifyLabel(issue_number, labelName, "DELETE");

try {
return await openai
Expand Down