Repository navigation
Migrate crow to nullkomma R package tooling - #50
Merged
Merged
Conversation
Initialize from the official template, use the shared brand, preserve GitHub Remotes and existing checks, and configure unlisted FlakeHub publishing. Replace Docker and Make orchestration while retaining Pages deployment and browser coverage; document upstream workarounds. Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
This was referenced Oct 26, 2026
Use the maxheld brand flake, retain explicit unlisted publication and Linux checks, and track macOS restoration in #52 and dataheld/nullkomma#255. Keep README package content unchanged and document only residual upstream integration gaps. Co-authored-by: Copilot App <[email protected]>
Remove the obsolete unlisted-publication upstream finding. Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
Record reproduction and workaround in dataheld/nullkomma#256. Co-authored-by: Copilot App <[email protected]>
Use nullkomma checks without the additional history-scanning publication gate. Co-authored-by: Copilot App <[email protected]>
This was referenced Oct 26, 2026
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
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.
Replace the Docker/Make development and CI stack with nullkomma's R-package and pkgdown support, preserving package behavior and substantive tests.
Approach
Remotes: dataheld/elf; pkg.lock pins current elf main at 50aa5444cdfe81ea8114e505174afbd603b1df57.Integration details
Cloudflare deployment runs only after successful main-branch CI and uses nullkomma's deploy app. It requires the Cloudflare zone, account/state-bucket variables and API/R2 secrets used by nullkomma. The app identity cannot inspect Actions variables or secrets (HTTP 403), so their presence and live deployment remain unverified.
UPSTREAM.md contains only remaining integration gaps. The project workflow selects Linux runners while macOS CI is deferred, and publishes the flake publicly. Browser-font and elf startup workarounds remain scoped to those demonstrated gaps.
Validation
The updated installed package passed all 46 assertions locally on macOS, with zero failures/warnings and only four existing fixture-setup skips. The branded pkgdown site and privacy page build, working-tree quality tasks pass, and formatting checks pass. All exported sandbox checks now pass on aarch64-darwin, aarch64-linux and x86_64-linux, including both Linux installed browser suites. One x86 browser preflight timed out during concurrent builds and passed when rerun alone. Remote CI and live deployment remain unverified.