Skip to content

Updated debugger launch profiles for VSCode. Fixes issue #164 - #207

Open
Thomas-Rowlands wants to merge 2 commits into
mainfrom
fix-issue-206
Open

Thomas-Rowlands wants to merge 2 commits into
mainfrom
fix-issue-206

Conversation

@Thomas-Rowlands

@Thomas-Rowlands Thomas-Rowlands commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor
🦋 Bug Fix

PR Description

Updates the launch.json profiles to run successfully following the PR #164 changes.

Related Issues or other material

Closes #164

Screenshots, example outputs/behaviour etc.

✅ Added/updated tests?

  • [] This PR contains relevant tests / Or doesn't need to per the below explanation
    No additional tests needed, as it updates debugger profiles it must be manually tested locally.

[optional] What gif best describes this PR or how it makes you feel?

alt_text

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
carrottransform
   __init__.py201525%16–17, 20, 22, 25, 28–30, 33–37, 39, 41
   _version.py5260%5–6
carrottransform/cli
   __init__.py00100% 
carrottransform/cli/subcommands
   __init__.py00100% 
   run.py1591789%92–93, 105–107, 132, 165–167, 291, 299, 334–337, 383, 393
   run_v2.py20480%55, 57–59
carrottransform/tools
   __init__.py40100% 
   args.py1462881%96–97, 106–109, 133, 150, 154, 163, 168, 178, 181, 186, 197–199, 206, 212, 214, 222, 230, 260, 273, 283, 288, 331, 335
   at_path.py80100% 
   concept_helpers.py21195%27
   core.py69297%118, 126
   date_helpers.py45589%24–25, 28, 65, 70
   db.py210100% 
   file_helpers.py532749%18–20, 35, 37–41, 50–52, 57–61, 76, 78–80, 83–85, 87, 91, 96
   logger.py120100% 
   mapping_types.py200100% 
   mappingrules.py238598%120, 377, 380–382
   metrics.py1394667%129–134, 136, 143–156, 159, 168, 174–179, 262, 264–270, 286–288, 291, 298, 300, 303, 349–350
   omopcdm.py1531392%45–47, 131, 155, 160, 172–175, 181, 199, 205
   orchestrator.py1751094%87–88, 98, 101, 118–120, 205, 370, 372
   outputs.py1766364%50, 73, 125, 151–152, 169–171, 174–177, 180–183, 186, 189–190, 192–196, 198, 201, 204, 207, 211–212, 215, 217, 219, 221–222, 225–227, 230–232, 241–242, 251–253, 267, 281, 283, 290, 292–295, 297, 309–310, 315–316, 318–320, 331
   person_helpers.py872374%17–18, 20–21, 23–24, 85, 98, 102, 133–134, 147, 160–163, 166–168, 170–171, 174–175
   record_builder.py1822288%26, 56, 71, 76, 79, 88, 100, 108–109, 183, 210, 259, 300–301, 315, 323, 327, 341, 348, 365, 392–393
   sources.py1555266%27, 47, 50–51, 54, 64, 114–115, 184–188, 190–191, 193–194, 196, 199–200, 203–205, 207–212, 214, 218–220, 222, 228, 230–231, 233–234, 236, 239–240, 243–245, 247–252, 254
   stream_helpers.py340100% 
   types.py640100% 
   validation.py30197%51
TOTAL203633683% 
Tests Skipped Failures Errors Time
87 1 💤 0 ❌ 0 🔥 9.670s ⏱️

@Thomas-Rowlands Thomas-Rowlands added the good first issue Good for newcomers label Oct 16, 2026
@Thomas-Rowlands Thomas-Rowlands self-assigned this Oct 16, 2026
@Thomas-Rowlands
Thomas-Rowlands marked this pull request as ready for review October 16, 2026 11:51

@AndrewThien AndrewThien left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Assumed that you already test the new launch profiles

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants