Skip to content

refactor(qc): remove ESGF branding from partner-course + QC projects (#1629 Phase 3a)#1640

Merged
jsboige merged 1 commit into
mainfrom
feature/genericize-esgf-1629-p3
May 28, 2026
Merged

refactor(qc): remove ESGF branding from partner-course + QC projects (#1629 Phase 3a)#1640
jsboige merged 1 commit into
mainfrom
feature/genericize-esgf-1629-p3

Conversation

@jsboige
Copy link
Copy Markdown
Owner

@jsboige jsboige commented May 27, 2026

Summary

  • Remove school-specific files from partner-course-quant-trading/ (ESGF_EXERCISES, UPLOAD_CHECKLIST, email template)
  • Rename ESGF class names in 9 QC strategy main.py files to generic names
  • Rewrite kit-transitoire/README.md in generic English
  • Update student-presentation template (remove ESGF/date references)
  • Fix starter template organization reference

Phase context

Test plan

  • grep -r ESGF partner-course-quant-trading/ returns 0 (excluding .ipynb and lean-workspace build artifacts)
  • All class names follow PEP 8 (no ESGF prefix)
  • 15 files changed: 3 deletions, 9 class renames, 3 content rewrites

Co-Authored-By: Claude Opus 4.7 noreply@anthropic.com

…1629 Phase 3a)

- Remove school-specific files (ESGF_EXERCISES, UPLOAD_CHECKLIST, email template)
- Rename ESGF class names in QC strategy main.py files (9 files)
- Rewrite kit-transitoire/README.md in generic English
- Update student-presentation template (remove ESGF/date references)
- Fix starter template organization reference

Phase 3b (next): docs/, scripts/, .claude/, notebooks ESGF text mentions

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@clusterManager-Myia clusterManager-Myia left a comment

Choose a reason for hiding this comment

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

[Hermes] — APPROVED

PR: refactor(qc): remove ESGF branding from partner-course + QC projects (#1629 Phase 3a)
SHA: 76699de

Checklist

  • Security scan: ✅ CLEAN (0 match)
  • Cross-repo impact: ✅ N/A (CoursIA docs/templates)
  • CI: ✅ 3/3 pass (staleness, Gitleaks, catalog drift)
  • Second reviewer: ✅ Non requis (15 fichiers mais +97/-483 = suppression/rename uniquement)

Verification

Diff = renaming classes (ESGFPrefix → GenericName), deleting 3 ESGF-specific MD files, updating README from FR→EN branding. No logic changes in main.py files (2-line class renames each). 5 project files touch only class name + docstring. Templates updated to remove org-specific references.

Clean de-branding. No concerns.

@jsboige jsboige merged commit c111d80 into main May 28, 2026
3 checks passed
@jsboige jsboige deleted the feature/genericize-esgf-1629-p3 branch May 29, 2026 23:06
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.

2 participants