Conversation
WalkthroughProject metadata and documentation updated to reflect version 0.2.0 release. Version bumped from 0.0.1a2, project description expanded to include Packmol and NetworkX support, repository URLs added to configuration, and Git clone instructions updated to include submodule initialization. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
uv.lockis excluded by!**/*.lock
📒 Files selected for processing (3)
README.md(1 hunks)docs/source/index.rst(1 hunks)pyproject.toml(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (12)
- GitHub Check: pytest (3.10, macos-15-intel)
- GitHub Check: pytest (3.11, windows-latest)
- GitHub Check: pytest (3.12, windows-latest)
- GitHub Check: pytest (3.11, macos-15-intel)
- GitHub Check: pytest (3.13, windows-latest)
- GitHub Check: pytest (3.13, macos-15-intel)
- GitHub Check: Build wheels on windows-latest
- GitHub Check: Build wheels on macos-latest
- GitHub Check: Build wheels on macos-15-intel
- GitHub Check: Build wheels on windows-11-arm
- GitHub Check: Build wheels on ubuntu-24.04-arm
- GitHub Check: Build wheels on ubuntu-latest
🔇 Additional comments (3)
README.md (1)
8-8: Title update accurately reflects expanded project scope.The updated title now correctly advertises Packmol and NetworkX support, which is already demonstrated in the existing documentation examples throughout the file.
docs/source/index.rst (1)
159-168: Installation instructions properly handle git submodule initialization.Clear, practical guidance with both proactive (--recursive flag) and reactive (git submodule update) approaches for users. This reduces friction for first-time contributors.
pyproject.toml (1)
3-4: Version and description updates consistent with release 0.2.0.The version bump to 0.2.0 aligns with the PR branch (v0.2.0) and the description now accurately reflects the expanded project scope across RDKit, ASE, Packmol, and NetworkX—matching updates in README and documentation.
| [project.urls] | ||
| Repository = "https://github.com/zincware/molify" | ||
| Releases = "https://github.com/zincware/molify/releases" | ||
| Discord = "https://discord.gg/7ncfwhsnm4" | ||
| Documentation = "https://molify.readthedocs.io/" |
There was a problem hiding this comment.
🧩 Analysis chain
Project URLs follow PyPI best practices.
The new [project.urls] section provides standard metadata for PyPI listings, improving project discoverability and user access to resources.
Please verify that the Discord invite link and documentation URL are active and current:
🌐 Web query:
Verify zincware molify project URLs on PyPI and GitHub
💡 Result:
Confirmed:
- PyPI (molify): https://pypi.org/project/molify/ [1].
- GitHub (source repo referenced in the project docs): https://github.com/zincware/rdkit2ase (repo for "molify"/rdkit2ase). [2]
Sources: PyPI project page for molify [1] and the zincware documentation which points to the GitHub repo (rdkit2ase) [2].
I need to gather more specific information. The initial search results show a potential repository mismatch (molify vs rdkit2ase) and don't verify the Discord or documentation URLs. Let me search for more details:
Repository URL is incorrect in pyproject.toml.
The package is installed via "pip install molify", but the actual GitHub repository is https://github.com/zincware/rdkit2ase, not https://github.com/zincware/molify. Update line 44:
Repository = "https://github.com/zincware/rdkit2ase"
The Discord invite link and ReadTheDocs documentation URL could not be verified through web searches and should be manually confirmed to ensure they are active and accessible.
🤖 Prompt for AI Agents
In pyproject.toml around lines 43 to 47, the Repository URL is incorrect (points
to /molify) and should be updated to the actual GitHub repo for the package;
change the Repository entry to "https://github.com/zincware/rdkit2ase". Also
verify the Discord invite and Documentation (ReadTheDocs) URLs—if you cannot
confirm they are valid, remove or replace them with verified links or add a TODO
comment to have the maintainer supply the correct URLs.
Summary by CodeRabbit
Documentation
Chores