git/GithubWorkflowTool
1
0
Fork 0
mirror of https://github.com/johndoe6345789/GithubWorkflowTool.git synced 2026-04-24 13:45:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix handleDoctor to work with QMap jobs structure

Browse Source 
- Add QFileInfo include
- Fix job iteration to use QMap::contains() for dependency checking
- Use proper QMap iterator for job traversal

Co-authored-by: johndoe6345789 <224850594+johndoe6345789@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2025-12-27 03:26:29 +00:00
parent a2ddedb744
commit 1d20f8f295
1 changed files with 6 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/cli/CommandHandler.cpp
View File

@@ -8,6 +8,7 @@
#include <QDebug>
#include <QProcess>
#include <QFile>
#include <QFileInfo>
namespace gwt {
namespace cli {
@@ -238,17 +239,11 @@ int CommandHandler::handleDoctor(const QStringList& args) {
// Check for job dependencies
bool hasValidDeps = true;
for (const auto& job : workflow.jobs) {
for (auto it = workflow.jobs.begin(); it != workflow.jobs.end(); ++it) {
const WorkflowJob& job = it.value();
if (!job.needs.isEmpty()) {
for (const QString& dep : job.needs) {
bool found = false;
for (const auto& otherJob : workflow.jobs) {
if (otherJob.id == dep) {
found = true;
break;
}
}
if (!found) {
if (!workflow.jobs.contains(dep)) {
hasValidDeps = false;
out << "✗ Error: Job '" << job.id << "' depends on non-existent job '" << dep << "'" << Qt::endl;
errors++;
@@ -260,8 +255,8 @@ int CommandHandler::handleDoctor(const QStringList& args) {
if (hasValidDeps && workflow.jobs.size() > 1) {
bool hasDeps = false;
for (const auto& job : workflow.jobs) {
if (!job.needs.isEmpty()) {
for (auto it = workflow.jobs.begin(); it != workflow.jobs.end(); ++it) {
if (!it.value().needs.isEmpty()) {
hasDeps = true;
break;
}