Skip to content

Fixed tbprofiler results_md path in services.json (fixes #638)#640

Merged
magdasmat merged 2 commits intoBU-ISCIII:developfrom
magdasmat:fix_tbprofiler_services_json
Mar 17, 2026
Merged

Fixed tbprofiler results_md path in services.json (fixes #638)#640
magdasmat merged 2 commits intoBU-ISCIII:developfrom
magdasmat:fix_tbprofiler_services_json

Conversation

@magdasmat
Copy link
Copy Markdown
Contributor

@magdasmat magdasmat commented Mar 17, 2026

Description

Fixed wrong file paths for delivery_md and results_md fields in the tbprofiler service entry in buisciii/templates/services.json.

  • delivery_md path: assets/reports/md/tbprofiler_delivery.mdassets/reports/md/tbprofiler.md
  • results_md path: assets/reports/results/tbprofiler_results.mdassets/reports/results/tbprofiler.md

Closes #638

PR checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (black and flake8).
  • If a new tamplate was added make sure:
    • Template's schema is added in templates/services.json.
    • Template's pipeline's documentation in assets/reports/md/template.md is added.
    • Results Documentation in assets/reports/results/template.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).
  • If you know a new user was added to the SFTP, make sure you added it to templates/sftp_user.json

@magdasmat magdasmat requested a review from victor5lm March 17, 2026 10:01
Copy link
Copy Markdown
Member

@Shettland Shettland left a comment

Choose a reason for hiding this comment

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

LGTM

@magdasmat magdasmat merged commit dc50e00 into BU-ISCIII:develop Mar 17, 2026
2 checks passed
@magdasmat magdasmat deleted the fix_tbprofiler_services_json branch March 17, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants