Skip to content

fix: align converted assessment report path - #38

Open
RKS (rksharma-owg) wants to merge 1 commit into
microsoft:mainfrom
rksharma-owg:fix/assessment-report-path
Open

RKS (rksharma-owg) wants to merge 1 commit into
microsoft:mainfrom
rksharma-owg:fix/assessment-report-path

Conversation

@rksharma-owg

Copy link
Copy Markdown

Summary

The assessment report converter instructed agents to write public report.json artifacts under .github/modernize/reports/, while the assessment and planning pipeline reads them from .github/modernize/assessment/reports/. This aligns the converter input contract, validation examples, and output contract with the consuming pipeline.

Validation

  • python3 -m unittest discover -s plugins/github-copilot-modernization/skills/assessment-report-converter/tests -v — 32 passed, 3 skipped because PowerShell was unavailable
  • Baseline/post-change agent pressure test — conflicting paths before; identical converter/planner paths after
  • bash -n for shipped assessment and telemetry shell scripts
  • jq -e . for all plugin JSON files
  • python3 -m py_compile for Python utility scripts
  • git diff --check upstream/main...HEAD
  • Fork CodeQL preflight for exact SHA b3a6d267b8103a3e205a317408e596ccfbaa5288 — Python and JavaScript/TypeScript passed: https://github.com/rksharma-owg/github-copilot-modernization/actions/runs/35555133933

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.

1 participant