Skip to content

Fix CI pipeline failures#798

Open
pavsoss wants to merge 5 commits into
Userunknown84:mainfrom
pavsoss:fix-ci-pipelines
Open

Fix CI pipeline failures#798
pavsoss wants to merge 5 commits into
Userunknown84:mainfrom
pavsoss:fix-ci-pipelines

Conversation

@pavsoss

@pavsoss pavsoss commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Description

This PR resolves multiple CI failures affecting backend validation, fork-based PR workflows, and the ML API Docker smoke test.

The changes clean up malformed backend code, restore greeting workflow functionality for fork PRs, and provide the required environment configuration for the ML API container during CI.

Changes

Rate Limiter Fix

  • Removed accidentally duplicated file contents from rateLimiter.js.
  • Restored a clean and valid module.exports object.
  • Ensured all required rate limiters are exported correctly.

PR Greeting Workflow

  • Updated the workflow trigger from pull_request to pull_request_target.
  • Allows the greeting action to comment on pull requests opened from forks.
  • Restores the expected greeting workflow behavior for external contributors.

ML API Docker Smoke Test

  • Added the required INTERNAL_SECRET environment variable to the Docker container startup command.
  • Uses a valid test-only secret for the CI environment.
  • Prevents ML API container startup and smoke test failures caused by missing configuration.

Testing

  • Verified the rate limiter module loads with valid exports.
  • Verified the workflow configuration includes the updated PR trigger.
  • Verified the ML API Docker smoke test receives the required INTERNAL_SECRET configuration.
  • Confirmed the changes are limited to the affected CI failures.

Closes #783

@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown

@pavsoss is attempting to deploy a commit to the Aditya Sharma's projects Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread backend/bulk_predict.py

from flask import Blueprint, current_app, jsonify, request, send_file

from api import limiter

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This limiter is removed as we need for prediction.

@Userunknown84 Userunknown84 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Fix the limit in bul_predict.py and ratelimiter i checked is ok a si matchec the code which is delete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix failing CI workflows and rateLimiter.js syntax error causing broken PR checks

2 participants