Skip to content
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

Updated build template #1

Merged
merged 6 commits into from
Mar 10, 2018
Merged

Updated build template #1

merged 6 commits into from
Mar 10, 2018

Conversation

gpampara
Copy link
Member

@gpampara gpampara commented Mar 8, 2018

This set of changes removes a whole bunch of duplication between the two different templates and consolidates them both into a single template.

A config file has been added for the user to change in order to adjust how the final document is produced. The build system has been replaced with latexmk instead of the Makefile approach.

gpampara added 6 commits March 8, 2018 10:42
* Use latexmk for all the builing of the document
* Transition to LuaLaTeX, from pdfLaTeX, as the former is the successor
* Template structure and content are now totally separated
* Template will import content and other config values using lua functions
* Mentioned config (`config.lua`) defined for values to be injected
  into the template at build time
@gpampara gpampara requested a review from stefanvds March 8, 2018 08:47
@gpampara
Copy link
Member Author

gpampara commented Mar 8, 2018

There are a bunch of updates to the way packages work, which is still to be worked on

@stefanvds
Copy link
Collaborator

Nice refactoring using lua to pull out config from the LaTeX doc. More refinement can be added over time, but This template is more configurable than the static LaTeX version.

@stefanvds stefanvds merged commit 2e1d344 into master Mar 10, 2018
@gpampara gpampara deleted the updated-build-template branch March 10, 2018 20:20
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