Skip to content

Conversation

@asmacdo
Copy link
Member

@asmacdo asmacdo commented Oct 15, 2025

Closes #89

This change means we can keep a list of dois in data/publications.yaml and publications.md can be automatically generated. (Currently done with a manual step, documented in the README)

This works as-is, remaining TODOs:

  • needs CI to automatically regenerate publications.md if data/publications.yaml changes and push a new change
    • Audit failure behavior, probably if any citations fail to render we should error out instead of just printing warnings
  • Audit format: we lost some formatting, ie no italics and doi links are longer. may need a post processing step or a different style from citeproc-py.
  • Alternative location for pyproject.toml? May not belong in the root, since overall this is not a python project.

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "uv run python scripts/generate_publications.py",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [
  "data/publications.yaml"
 ],
 "outputs": [
  "content/about/publications.md"
 ],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@netlify
Copy link

netlify bot commented Oct 15, 2025

Deploy Preview for repronim ready!

Name Link
🔨 Latest commit 7514818
🔍 Latest deploy log https://app.netlify.com/projects/repronim/deploys/68efadb06cf6810008eb5e2a
😎 Deploy Preview https://deploy-preview-421--repronim.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@memartone memartone requested a review from dnkennedy October 21, 2025 20:09
@dnkennedy
Copy link
Contributor

Couple comments, related to the differences between this and the original, and my opinion:

  • I would like the article title and journal name to be italicized,
  • Current shows 1 author initial, original showed at least 2 (perhaps more?); I think I like the multiple initials (ie. D.N. Kennedy as opposed to D. Kennedy)
  • There seems to be an extra period '.' after the authors, before the title, I think that should not be there.

I can try to invoke these updates.

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.

Make "publications" produced, not just entered and associate with ppl

2 participants