Repository navigation
Add In Pool column to candidates table - #44
Merged
Merged
Conversation
Adds a ninth column to the autogenerated candidates README table that shows ✓ for candidates already imported into lean-pool. The merged status is sourced from the optional `source.github_repo` field on each entry in `LeanPool/projects.yml`, joined to the candidate manifest by canonical owner/name key. Wires the path through the `render` CLI behind a `--projects-yml` flag (defaulting to `LeanPool/projects.yml`) and re-renders the README with the new column.
Contributor
|
LLM review skipped: PR touches no |
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.
Summary
In Poolcolumn to the autogenerated candidates README table that renders ✓ for candidates already imported into lean-pool.source.github_repofield on entries inLeanPool/projects.yml, joined to the manifest by canonicalowner/namekey (existingcanonical_key/package_keymachinery).renderCLI behind a new--projects-ymlflag (defaults toLeanPool/projects.yml).candidates/README.md. Every cell is currently blank because noprojects.ymlentry has agithub_reposource yet — future imports flip themselves to ✓ by adding the field.The existing
projects.ymlschema check inquality.pyalready accepts dict sources containing arbitrary extra keys (it requires exactly one of{arxiv, doi, url}to be present), so no schema change was needed.