Skip to content

Use conda-forge rdkit-postgresql in README; drop Apple-silicon caveats - #180

Merged
skearnes merged 2 commits into
mainfrom
readme-rdkit-conda-forge
May 24, 2026
Merged

Use conda-forge rdkit-postgresql in README; drop Apple-silicon caveats#180
skearnes merged 2 commits into
mainfrom
readme-rdkit-conda-forge

Conversation

@skearnes

@skearnes skearnes commented May 22, 2026

Copy link
Copy Markdown
Member

Summary

  • rdkit-postgresql is available on Apple silicon via conda-forge (already how CI installs it in tests.yml), so the special-case instructions are no longer needed.
  • Switches both READMEs from -c rdkit to -c conda-forge to match CI.

Test plan

  • Skim rendered READMEs.

🤖 Generated with Claude Code

Greptile Summary

Both READMEs are updated to replace the -c rdkit channel with -c conda-forge for installing rdkit-postgresql, and the Apple silicon / x86_64-only caveats are dropped. The change is consistent with .github/workflows/tests.yml, which already uses conda install -c conda-forge rdkit-postgresql against both ubuntu-latest and macos-14 runners.

  • README.md: Removes the "Apple silicon: use conda install postgresql instead" comment and switches to the conda-forge channel.
  • ord_interface/api/README.md: Removes the "rdkit-postgresql is currently only available on x86_64" note and switches to the conda-forge channel.

Confidence Score: 5/5

Documentation-only change; the new instructions exactly match what CI already runs successfully on both Linux and Apple silicon.

Both README files now reflect the conda-forge channel that CI has used all along, and the removed caveats are confirmed obsolete by the macOS CI matrix. No code paths are touched.

No files require special attention.

Important Files Changed

Filename Overview
README.md Removes Apple silicon caveat and switches channel from -c rdkit to -c conda-forge, matching CI.
ord_interface/api/README.md Removes the x86_64-only note and switches channel from -c rdkit to -c conda-forge.

Reviews (1): Last reviewed commit: "Merge branch 'main' into readme-rdkit-co..." | Re-trigger Greptile

rdkit-postgresql is available on Apple silicon via conda-forge (already
how CI installs it), so the special-case instructions are no longer
needed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@skearnes
skearnes requested a review from bdeadman May 22, 2026 01:36
@skearnes
skearnes enabled auto-merge (squash) May 24, 2026 00:42
@skearnes
skearnes merged commit f08d69e into main May 24, 2026
16 checks passed
@skearnes
skearnes deleted the readme-rdkit-conda-forge branch May 24, 2026 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants