JOJ/JOJ3-config-generator
27
1
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

dev #10

Merged
李衍志523370910113 merged 238 commits from dev into master 2025-03-05 16:20:39 +08:00
Conversation 16 Commits 238 Files Changed 42 +1 -4
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9c20c6b377 - Show all commits

5
joj3_config_generator/processers/task.py
View File

@ -14,7 +14,6 @@ def get_conf_stage(
conf_stage = result.StageDetail(
name=task_stage.name,
# group is determined by adding between "[]" in the name of the task
jon-lee marked this conversation as resolved Outdated
李衍志523370910113 commented 2025-03-03 11:12:45 +08:00
Outdated
Review
Copy Link

BTW, is this outdated?

BTW, is this outdated?
张泊明518370910136 commented 2025-03-03 11:17:54 +08:00
Outdated
Review
Copy Link

Never heard about this rule.

Never heard about this rule.
李衍志523370910113 commented 2025-03-03 11:19:19 +08:00
Outdated
Review
Copy Link

@manuel what would be the current intended rule for group?

@manuel what would be the current intended rule for `group`?
李衍志523370910113 commented 2025-03-03 16:18:13 +08:00
Outdated
Review
Copy Link

seems current strategy is fine, resolved.

seems current strategy is fine, resolved.
# FIXME: this is probably outdated
group=(
match.group(1)
if (match := re.search(r"\[([^\[\]]+)\]", task_stage.name or ""))
@ -110,9 +109,7 @@ def fix_keyword(
def fix_result_detail(
task_stage: task.Stage, conf_stage: result.StageDetail, parser: str
) -> None:
result_detail_parser = next(
p for p in conf_stage.parsers if p.name == "result-detail"
)
result_detail_parser = next(p for p in conf_stage.parsers if p.name == parser)
show_files = []
if task_stage.result_detail.stdout:
show_files.append("stdout")