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

✨ Add Docker support; Use FM parser w/ ruamel.yaml #68

Merged
merged 1 commit into from
Aug 13, 2020
Merged

✨ Add Docker support; Use FM parser w/ ruamel.yaml #68

merged 1 commit into from
Aug 13, 2020

Conversation

jmuchovej
Copy link
Contributor

  • Adding docker-compose support.
  • Leverage hugo new [path] --kind publication so changes to archetype
    are reflected.
  • "Implement" (copied from personal repo) a Front-Matter parser to
    enable greater code readability and proper YAML support.
    • Add ruamel.yaml as a dependency for round-trip YAML parsing with
      commend support.
  • Chunked bibtex-related code into import_bibtex for improved
    readability.

- Adding `docker-compose` support.
- Leverage `hugo new [path] --kind publication` so changes to archetype
  are reflected.
- "Implement" (copied from personal repo) a Front-Matter parser to
  enable greater code readability and proper YAML support.
  - Add `ruamel.yaml` as a dependency for round-trip YAML parsing with
    commend support.
- Chunked `bibtex`-related code into `import_bibtex` for improved
  readability.
@gcushen
Copy link
Collaborator

gcushen commented Aug 13, 2020

Thanks for your contribution! Great use of ruamel.yaml :D

Feel free to consider helping with #49 if you find time as it's quite closely related.

@arichardson arichardson mentioned this pull request Sep 30, 2020
gcushen pushed a commit that referenced this pull request Oct 4, 2020
Bug introduced by #68. Dry run should not create publication files.

Fix #68
See #77
gcushen pushed a commit that referenced this pull request Oct 4, 2020
Fixes bug introduced by #68 with expecting publication files to be generated even in dry run.

Fixes bug introduced by #68 with file not closed.

Fix 56634da
Fix #68
See #75
@gcushen gcushen mentioned this pull request Oct 20, 2020
gcushen added a commit that referenced this pull request Jan 9, 2021
A leftover artifact from the #68 refactor

Fix #68
Fix #89
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.

3 participants