Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds theme support for Huazhong University of Science and Technology (HUST) to the collegeBeamer template system, using PDF format for assets instead of PNG.
Key Changes:
- Added HUST theme option with custom colors and PDF-based logos/background
- Updated example in pre.tex to demonstrate HUST theme usage
- Introduced latexmkrc build configuration file
Reviewed changes
Copilot reviewed 3 out of 6 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| collegebeamer.sty | Adds HUST theme option with RGB color scheme and PDF asset paths |
| pre.tex | Updates documentation, example usage, and metadata to showcase HUST theme |
| .latexmkrc | Adds new LaTeX build configuration for XeLaTeX compilation with custom settings |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| $force_mode = 0; | ||
|
|
||
| # Ensure miniconda bin is on PATH for runtime tools (e.g., minted executable, pygmentize) | ||
| $ENV{'PATH'} = '$YOUR_CUSTOM_FILE_PATH:' . $ENV{'PATH'}; |
There was a problem hiding this comment.
This line contains a placeholder string that should not be committed to the repository. The literal string '$YOUR_CUSTOM_FILE_PATH' will be added to the PATH environment variable, which is incorrect. Either replace this with an actual path if needed, or remove this line entirely if custom PATH modification is not required for the basic functionality of the template.
| # Use XeLaTeX by default (pdf_mode=5) | ||
| $pdf_mode = 5; |
There was a problem hiding this comment.
This new configuration file lacks documentation explaining its purpose and usage. Consider adding a comment at the top of the file explaining when and why users should use this latexmkrc configuration, especially since the README doesn't mention it and other users may already have their own latexmkrc configurations.
Due to capability reasons, I use PDF format for the background and logo.