Skip to content

Fix: handle rate limit error #58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AlexSim93 merged 2 commits into from
Dec 19, 2024
Merged

Fix: handle rate limit error #58

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bc019fd
Fix: handle rate limit error
AlexSim93 Dec 18, 2024
2478a80
Fix: action version
AlexSim93 Dec 19, 2024
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
18 changes: 14 additions & 4 deletions build/index.js
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "pull-request-analytics-action",
"version": "4.4.0",
"version": "4.4.2",
"description": "Generates detailed PR analytics reports within GitHub, focusing on review efficiency and team performance.",
"main": "build/index.js",
"scripts": {
Expand Down
35 changes: 21 additions & 14 deletions src/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,7 @@ import {
validate,
} from "./common/utils";
import { getRateLimit } from "./requests/getRateLimit";
import {
sendActionError,
sendActionRun,
} from "./analytics";
import { sendActionError, sendActionRun } from "./analytics";

async function main() {
try {
Expand All @@ -34,11 +31,17 @@ async function main() {
);
return;
}
const rateLimitAtBeginning = await getRateLimit();
console.log(
"RATE LIMIT REMAINING BEFORE REQUESTS: ",
rateLimitAtBeginning.data.rate.remaining
);
try {
const rateLimitAtBeginning = await getRateLimit();
console.log(
"RATE LIMIT REMAINING BEFORE REQUESTS: ",
rateLimitAtBeginning.data.rate.remaining
);
} catch (error) {
console.log(
"Rate limit could not be retrieved at the beginning of the action"
);
}

const ownersRepos = getOwnersRepositories();
const organizationsRepos = await getOrganizationsRepositories();
Expand Down Expand Up @@ -93,11 +96,15 @@ async function main() {
console.log("Calculation complete. Generating markdown.");
await createOutput(preparedData);

const rateLimitAtEnd = await getRateLimit();
console.log(
"RATE LIMIT REMAINING AFTER REQUESTS: ",
rateLimitAtEnd.data.rate.remaining
);
try {
const rateLimitAtEnd = await getRateLimit();
console.log(
"RATE LIMIT REMAINING AFTER REQUESTS: ",
rateLimitAtEnd.data.rate.remaining
);
} catch (error) {
console.log("Rate limit could not be retrieved at the end of the action");
}
} catch (error) {
sendActionError(error as Error);
throw error;
Expand Down
Loading