Closed drupol closed 1 year ago
I really appreciate the hard work you're putting in this <3
Thanks mate, it's been a while I wanted to do that... but I was lacking knowledge in LaTeX and Nix... and I think now it is the time :) I'll finish this today after work.
Awesome, because I haven't touched this stuff in years :) Go for it!
Please make sure all those indentations now in code snippets do not render as extra whitespace.
Please make sure all those indentations now in code snippets do not render as extra whitespace.
There is a configuration file .latexindent.yaml
which prevent the verbatim environment to be modified. So it's all good.
<3
I think this PR is ready, let me know if you're ok with it :)
I know it's a lot to review.
That said, I tried to be very cautious and tidy with commits.
The easiest way to review such PR is to click on each commit and understand what it's doing.
Let me know if something needs to be changed.
Hi,
The motivation of this PR is reproducibility and accessibility. I want to make sure that anyone will be able to build the PDFs using the same mechanism and workflow and that everyone will use the same version of dependencies to better track potential issues.
I've been working with Nix and LaTeX at work and for personal stuff lately and I'm now ready to upgrade this project with the best practices and technologies. The idea here is to make sure we don't break the build system based on Nix and provide a better development environment for contributors.
Status:
numtide/flake-utils
toflake-parts
alejandra
nixos-21.11
tonixpkgs-unstable
, frompython-3.8
topython-3.11
.envrc
to load the development environment instantlyprettier
at each pull-requestprettier
latexindent.pl
Collateral things:
minted
LaTeX package has been encountered, see https://github.com/gpoore/minted/issues/353, a workaround(https://github.com/hmemcpy/milewski-ctfp-pdf/pull/306/commits/91779a5d1c3ac4f52e19dd4f1165fe0f4f2f8d95) has been put in place and it's working.pygments-style-github
innixpkgs
, see https://github.com/NixOS/nixpkgs/pull/214015