Skip to content

Glossary generator with apa lookup#595

Open
kaysiz wants to merge 15 commits intomasterfrom
ks-refactor-glossary-generator
Open

Glossary generator with apa lookup#595
kaysiz wants to merge 15 commits intomasterfrom
ks-refactor-glossary-generator

Conversation

@kaysiz
Copy link
Collaborator

@kaysiz kaysiz commented Jan 26, 2026

Description

Fixes # (issue)

Type of Change

  • Documentation/Contents/grammar update
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Tested Locally
  • Manual review / previewed on staging.forrt.org content/webpage changes
  • Not Tested yet

Checklist for Content Editors and Non-Developers

  • The content is clear, accurate, and follows community guidelines.
  • All updated content has been previewed on the staging site.
  • All links, references, and formatting have been checked for correctness.
  • The change aligns with the overall style and communication goals.
  • No broken links in text/content

Checklist for Developers:

  • I have attempted to stay aligned to related code in this repository rather than reinventing the wheel.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

Additional Notes

@github-actions github-actions bot added content related Relevant to website content cicd Relevant to GitHub workflows labels Jan 26, 2026
@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

👍 All image files/references (if any) are in webp format, in line with our policy.

@kaysiz kaysiz requested a review from LukasWallrich January 27, 2026 08:47
@kaysiz kaysiz self-assigned this Jan 27, 2026
@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an APA-reference lookup workflow and updates glossary generation to use it when producing glossary entry files.

Changes:

  • Added content/glossary/apa_lookup.json as a citation-key → APA-formatted reference lookup.
  • Refactored content/glossary/_create_glossaries.py to pull glossary data from a Google Sheets CSV export and format references via the APA lookup.
  • Added a Node.js utility (bibtex_to_apa/) and CI workflow steps to generate/update the APA lookup during data processing.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
content/glossary/apa_lookup.json New citation-key → APA reference mapping used during glossary generation.
content/glossary/_create_glossaries.py New glossary generation pipeline that reads a sheet export and formats references using the APA lookup.
bibtex_to_apa/package.json Adds dependencies for converting BibTeX to APA via citation-js.
bibtex_to_apa/package-lock.json Lockfile for the bibtex→APA conversion tool dependencies.
bibtex_to_apa/bibtex_to_apa.js Node script to fetch BibTeX and emit apa_lookup.json.
.github/workflows/data-processing.yml Installs Node deps and runs the bibtex→APA generation step in CI.
Files not reviewed (1)
  • bibtex_to_apa/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

1 similar comment
@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

kaysiz and others added 2 commits February 10, 2026 20:36
@kaysiz kaysiz marked this pull request as ready for review February 10, 2026 19:36
@kaysiz kaysiz requested a review from a team as a code owner February 10, 2026 19:36
@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Feb 10, 2026

Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-02-13 12:52:55 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@LukasWallrich
Copy link
Contributor

@copilot the references generated by this PR look like the attached. Can you help figure out what is wrong?
image

Copy link
Contributor

Copilot AI commented Feb 11, 2026

@LukasWallrich I've opened a new pull request, #639, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd Relevant to GitHub workflows content related Relevant to website content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants