-
Notifications
You must be signed in to change notification settings - Fork 1
resolves 4 and 5: add support for devcontainer and add poster template #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ChkTeX Action
LGTM! 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for a devcontainer setup and updates CI workflows to include linting and LaTeX poster compilation while closing issue #4.
- Added a lint GitHub Actions workflow for code quality checks.
- Updated the LaTeX workflow to compile and manage poster artifacts.
- Introduced a new devcontainer configuration for LaTeX development.
Reviewed Changes
Copilot reviewed 4 out of 8 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/lint.yml | New workflow to run linting checks using chktex-action. |
| .github/workflows/latex.yml | Adds steps for poster compilation and updates artifact naming. |
| .devcontainer/docker-compose.yml | Provides a devcontainer configuration for LaTeX development. |
Files not reviewed (4)
- .devcontainer/Dockerfile: Language not supported
- .devcontainer/devcontainer.json: Language not supported
- .vscode/settings.json: Language not supported
- poster.template.tex: Language not supported
Comments suppressed due to low confidence (1)
.devcontainer/docker-compose.yml:4
- [nitpick] The service name 'vscode' is ambiguous given the container is used for LaTeX development. Consider renaming it to a name that reflects its purpose more clearly.
vscode:
| # Shell history persistence | ||
| - ~/.zsh_history:/root/.zsh_history:z | ||
| environment: | ||
| - TZ= |
Copilot
AI
Mar 31, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The environment variable 'TZ' is set to an empty value, which might lead to unexpected timezone behavior. Provide a valid timezone or remove the variable if it's not needed.
| - TZ= |
|
great work @prudhomm ! |
|
The design is not quite finished yet. We should probably put a footline: a qr code at the bottom and space for extra information. What do you think? |
Sure, good idea |
…e with new features and styling add footline in poster add poster template #5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ChkTeX Action
There are issues with your code. 😔 Check the annotations under the Files changed tab.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ChkTeX Action
There are issues with your code. 😔 Check the annotations under the Files changed tab.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ChkTeX Action
There are issues with your code. 😔 Check the annotations under the Files changed tab.
…e and author formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ChkTeX Action
There are issues with your code. 😔 Check the annotations under the Files changed tab.
…nd poster template
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ChkTeX Action
There are issues with your code. 😔 Check the annotations under the Files changed tab.
…LaTeX workflow for xelatex compilation
…guide, and CI/CD pipeline details
abuttari
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me.
Uh oh!
There was an error while loading. Please reload this page.