Skip to content

Update GH/RDD Page automation to include I&T reports#7

Merged
jordanpadams merged 5 commits into
mainfrom
claude/doc/add-int-reports
Jun 17, 2026
Merged

Update GH/RDD Page automation to include I&T reports#7
jordanpadams merged 5 commits into
mainfrom
claude/doc/add-int-reports

Conversation

@c-suh

@c-suh c-suh commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

🗒️ Summary

  • Checks build release number
  • If release 16+ and not current release, add links for I&T reports
  • If release 16+ and current release, check for time window (i.e., June, December - build release months) or if flag is enabled to add links for I&T reports

🤖 AI Assistance Disclosure

  • No AI assistance used
  • AI used for light assistance (e.g., suggestions, refactoring, documentation help, minor edits)
  • AI used for moderate content generation (AI generated some code or logic, but the developer authored or heavily revised the majority)
  • AI generated substantial portions of this code

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

  • Documentation: README, Wiki, or inline documentation (Sphinx, Javadoc, Docstrings) have been updated to reflect these changes.
  • Issue Traceability: The PR is linked to a valid GitHub Issue
  • PR Title: The PR title is "user-friendly" clearly identifying what is being fixed or the new feature being added, that if you saw it in the Release Notes for a tool, you would be able to get the gist of what was done.

Security & Quality

  • SonarCloud: Confirmed no new High or Critical security findings.
  • Secrets Detection: Verified that the Secrets Detection scan passed and no sensitive information (keys, tokens, PII) is exposed.
  • Code Quality: Code follows organization style guidelines and best practices for the specific language (e.g., PEP 8, Google Java Style).

Testing & Validation

  • Test Accuracy: Verified that test data is accurate, representative of real-world PDS4 scenarios, and sufficient for the logic being tested.
  • Coverage: Automated tests cover new logic and edge cases.
  • Local Verification: (If applicable) Successfully built and ran the changes in a local or staging environment.

Maintenance

  • Backward Compatibility: Confirmed that these changes do not break existing downstream dependencies or API contracts (or that breaking changes are clearly documented).

@c-suh c-suh requested a review from a team as a code owner June 13, 2026 06:51
@c-suh c-suh requested a review from Copilot June 13, 2026 06:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.py parsing to infer the current release and threaded that into summary generation; added --add-int-reports flag 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.

Comment thread src/lasso/reports/gh_pages/summary.py Outdated
Comment thread src/lasso/reports/gh_pages/summary.py Outdated
Comment thread src/lasso/reports/gh_pages/build_summaries.py Outdated
@c-suh c-suh requested a review from nutjob4life June 15, 2026 15:54
Comment thread src/lasso/reports/gh_pages/summary.py Outdated

@nutjob4life nutjob4life 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.

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!

Comment thread src/lasso/reports/gh_pages/summary.py Outdated
Comment thread src/lasso/reports/gh_pages/build_summaries.py Outdated
Comment thread src/lasso/reports/gh_pages/build_summaries.py Outdated
c-suh and others added 4 commits June 16, 2026 11:49
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
@c-suh c-suh requested a review from nutjob4life June 16, 2026 21:53

@nutjob4life nutjob4life 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.

Code deltas reviewed: ✓
Visual inspection: ✓
Testing: skipped
Approval: ✅
Thanks @c-suh!

@jordanpadams jordanpadams merged commit f4b08ba into main Jun 17, 2026
4 checks passed
@c-suh c-suh mentioned this pull request Jun 25, 2026
14 tasks
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.

4 participants