Skip to content

Conversation

@emyller
Copy link
Contributor

@emyller emyller commented Jan 7, 2026

Seemingly this outdated configuration has been failing CI for a while.

Changes

  • Rely on task_processor package provided by flagsmith-common.

@emyller emyller self-assigned this Jan 7, 2026
@emyller emyller requested a review from a team as a code owner January 7, 2026 19:23
@emyller emyller requested review from khvn26 and removed request for a team January 7, 2026 19:23
@vercel
Copy link

vercel bot commented Jan 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Review Updated (UTC)
docs Ignored Ignored Preview Jan 7, 2026 8:33pm
flagsmith-frontend-preview Ignored Ignored Preview Jan 7, 2026 8:33pm
flagsmith-frontend-staging Ignored Ignored Preview Jan 7, 2026 8:33pm

@emyller emyller added api Issue related to the REST API dependencies Pull requests that update a dependency file labels Jan 7, 2026
@github-actions github-actions bot added fix and removed dependencies Pull requests that update a dependency file labels Jan 7, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-e2e:pr-6493 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-6493 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-api-test:pr-6493 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith:pr-6493 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-api:pr-6493 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-6493 Finished ✅ Results

matthewelwell
matthewelwell previously approved these changes Jan 7, 2026
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.08%. Comparing base (42e4fdc) to head (6fede76).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6493   +/-   ##
=======================================
  Coverage   98.08%   98.08%           
=======================================
  Files        1296     1296           
  Lines       46616    46616           
=======================================
  Hits        45724    45724           
  Misses        892      892           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@emyller emyller merged commit 4a6ee45 into main Jan 7, 2026
29 checks passed
@emyller emyller deleted the fix/remove-task-processor-dependency branch January 7, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Issue related to the REST API fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants