Skip to content

Fix #1155: Refactor TaskExecutor to reduce complexity and improve maintainability #1239

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

Open
trivikramkalagi91-commits wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Fix #1155: Refactor TaskExecutor to reduce complexity and improve maintainability #1239

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
72d0f33
fix(ui): resolve layout overlap between TIME_MATRIX and DELETE_RECORD…
trivikramkalagi91-commits Jun 14, 2026
0a8e88e
refactor(executor): decompose execute_task and add unit tests
trivikramkalagi91-commits Jun 23, 2026
989ac15
Merge upstream/main into fix/layout-overlap-906-refactor
trivikramkalagi91-commits Jun 23, 2026
7e46a2b
trigger ci
trivikramkalagi91-commits Jun 23, 2026
f58f79c
fix(reports): rename client-side PDF button to Quick PDF to avoid uni…
trivikramkalagi91-commits Jun 23, 2026
6807693
test(backend): fix database workflow version and risk scoring tests f…
trivikramkalagi91-commits Jun 23, 2026
27b78e6
ci: fix frontend-checks required status check mapping for matrix
trivikramkalagi91-commits Jun 23, 2026
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
17 changes: 16 additions & 1 deletion .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ jobs:
if: steps.run_benchmarks.outcome == 'failure'
run: |
echo "::warning::Performance benchmark thresholds exceeded or benchmarks failed to run. Check the job logs for details."
frontend-checks:
frontend-run-checks:
needs: [detect-changes, formatting-hygiene]
if: |
always() &&
Expand Down Expand Up @@ -304,3 +304,18 @@ jobs:
run: npm run test
- name: Build frontend
run: npm run build

frontend-checks:
needs: [frontend-run-checks]
if: always()
runs-on: ubuntu-latest
steps:
- name: Check matrix results
run: |
if [ "${{ needs.frontend-run-checks.result }}" = "success" ] || [ "${{ needs.frontend-run-checks.result }}" = "skipped" ]; then
echo "Frontend checks completed successfully or skipped"
exit 0
else
echo "Frontend checks failed"
exit 1
fi
Loading
Loading