Streamline portfolio repository contents - #103
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR streamlines the portfolio repository by removing legacy cleanup/build scripts and adding a CI-safe validator to ensure the published project inventory and local documentation links remain consistent as the repository evolves.
Changes:
- Added a standard-library Python validator plus a GitHub Actions workflow to enforce portfolio artifact and link integrity in CI.
- Updated portfolio-facing documentation (README + resume + PDF export instructions) to reflect a four-project layout including the RAG Production Pipeline.
- Removed outdated cleanup notes and deprecated PDF/report build scripts.
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/validate_portfolio.py | New CI-friendly validator for required report/PDF artifacts and local index link integrity. |
| .github/workflows/validate-portfolio.yml | Adds an Actions workflow to run the validator on PRs and pushes to main. |
| README.md | Updates portfolio presentation (adds RAG project, validation badge, counts/last-updated metadata). |
| Resume_Derek_Lankeaux.md | Updates summary/achievements and adds the RAG Production Pipeline entry. |
| PDF_EXPORT.md | Replaces old Pandoc/XeLaTeX flow with the canonical generate_publication_pdfs.py process and documents CI validation. |
| scripts/build_reports_pdf.sh | Removes deprecated Pandoc/XeLaTeX-based PDF build script. |
| professionalize_reports.py | Removes legacy report-professionalization script (previously produced combined PDFs). |
| FINAL_CLEANUP_SUMMARY.md | Removes obsolete cleanup planning document. |
| CLEANUP_SUMMARY.md | Removes obsolete consolidation note. |
| cleanup_script.sh | Removes obsolete cleanup automation script that included branch deletion/push guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Owner
Author
|
good |
1 similar comment
Owner
Author
|
good |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Owner
Author
|
good |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.