Conversation
- DESCRIPTION: URL, BugReports, Additional_repositories → openalexPro org - _pkgdown.yml: site URL → openalexpro.github.io - README.md: all badge/link URLs updated to openalexPro org - CI workflows: extra-repositories → openalexpro.r-universe.dev Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… live openalexpro.r-universe.dev does not exist yet (packages.json needs to reach main first). Temporarily keep rkrug.r-universe.dev as the install source; will switch once the org universe is built. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…r-universe.dev rkrug.r-universe.dev dropped openalexPro after the org transfer. Use github::openalexPro/openalexPro in CI extra-packages as bridge until openalexpro.r-universe.dev finishes its first build. Additional_repositories updated to openalexpro.r-universe.dev. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
chore: move repo to openalexPro org
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add tests for the one-shot export wrapper, Pandoc HTML/single-file rendering, conversion error paths, and the Zotero uploader (validation plus mocked success/failure/network-error paths). Package coverage rises from ~70% to ~86%. Writing the wrapper tests surfaced two bugs that left corpus_export_via_pandoc() non-functional: - it called corpus_to_csljson() with positional args that misaligned with the project_dir-first signature; now uses named args - it pre-created the temp CSL dir, which corpus_to_csljson() then refused to overwrite; the path is now only reserved, not created Also reflow over-long lines in the R source for lintr/goodpractice. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
# Conflicts: # .github/workflows/R-CMD-check.yaml # .github/workflows/pkgdown.yaml # .github/workflows/test-coverage.yaml # DESCRIPTION
…ectory The wrapper passed the CSL JSON *directory* to csljson_convert_pandoc(), which for directory input treats `output` as a directory and writes chunk_*.bib into it. So `corpus_export_via_pandoc(output = "corpus.bib")` created a *directory* named corpus.bib rather than the single file its docs promise. The test only caught this on Windows, where a directory reports size 0 (vs. non-zero on macOS/Linux). Merge the chunked CSL JSON into a single array first, then run the single-file conversion, and return the actual created path (the previous `normalizePath(output)` was wrong when an extension was appended). Tests now assert the result is a file, not a directory, on all platforms. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Add Zenodo DOI, CI fixes, and raise test coverage to 86%
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Promotes dev → main after all CI passed on PR #4.
Includes org URL updates, pkgdown site URL, CI workflow fixes
(github::openalexPro/openalexPro bridge until r-universe is live).
🤖 Generated with Claude Code