Skip to content

Minor refactoring of generate.sh #82

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

oliverkwebb
Copy link

I hope to get the build infrastructure in a flexible enough state eventually so that the book can be compiled into different formats (mainly, PDF and single-page HMTL)

Changes Made:

  • Remove significant whitespace in generate.sh
  • Give generate.sh executable permissions
  • Re-run generate.sh, pushing a few previous changes to the generated html
  • sed command | sed command -> sed "command; command", unnecessary $ in shell math

P.S.
Right now, the filenames for the chapters are being stored in a simple text file, and the titles for the chapters are being extracted using sed, if that file was a YAML file, which stored the filenames and titles in a list, and the templating could use it to reference the previous/next chapters (with pandoc's $for$). It would remove the need for most of the shell code here (at the cost of the file being harder to parse)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant