fix: refresh rxiv-maker citation authors - #314
Conversation
🔗 Link Check FailedSome links in the documentation are broken: Summary
Errors per inputErrors in README.md
Errors in tests/visual/README.md
Redirects per inputRedirects in CODE_OF_CONDUCT.md
Redirects in docs/google_colab.md
Redirects in README.md
💡 Tip: Check the full report artifact for details. |
Code ReviewReviewed the diff for #314 (author refresh for the canonical Rxiv-Maker citation, v1.22.2). Overall: small, well-scoped fix. The refresh logic ( Minor issues
Security / performanceNo concerns — this is a self-citation string change plus a regex-based find/replace against a local file. No new attack surface (path handling unchanged from prior version, still resolves Test coverageGood — new integration test covers the "refresh stale entry" path end-to-end, and existing unit/integration tests were updated to reflect the new author list including the negative assertion that the old string is gone. |
Summary
Verification
uv run --with pytest pytest tests/integration/test_citation_injection.py -quv run --with ruff ruff check src/rxiv_maker/utils/citation_utils.py tests/integration/test_citation_injection.pyuv build