Update GH/RDD Page automation to include I&T reports#7
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the GH Pages build summary generation to optionally include Integration & Testing (I&T) report links for Build 16+ and to detect the “current release” by parsing conf.py, enabling different behavior for current vs past builds.
Changes:
- Added version parsing + conditional insertion of I&T PDF links into the RST introduction for Build 16+.
- Added
conf.pyparsing to infer the current release and threaded that into summary generation; added--add-int-reportsflag to force inclusion for the current build. - Minor text/comment cleanup in the RST generation path.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/lasso/reports/gh_pages/summary.py | Adds version parsing + logic to include I&T report links in RST summaries, with special handling for the current build. |
| src/lasso/reports/gh_pages/build_summaries.py | Determines current release from conf.py, passes it into summary generation, and adds a CLI flag to force I&T report inclusion. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
nutjob4life
left a comment
There was a problem hiding this comment.
Looks good; just a little "seasonal concern" with time of run versus time of what artifacts get produced. Also see a couple comments about logging calls with % placehoders and the encoding for open, otherwise super close!
Per review comments from nutjob4life and copilot from PR7 Part 1 of 3
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Also tweak order of checks/statements for dev and current release
nutjob4life
left a comment
There was a problem hiding this comment.
Code deltas reviewed: ✓
Visual inspection: ✓
Testing: skipped
Approval: ✅
Thanks @c-suh!
🗒️ Summary
🤖 AI Assistance Disclosure
Estimated % of code influenced by AI: 95%
⚙️ Test Data and/or Report
♻️ Related Issues
NASA-PDS/nasa-pds.github.io#85
🤓 Reviewer Checklist
Reviewers: Please verify the following before approving this pull request.
Documentation and PR Content
Security & Quality
Testing & Validation
Maintenance