Adapt to rocq-prover/rocq#21955 (rm coq-core libraries)#215
Conversation
📝 WalkthroughWalkthroughThis pull request updates build and package configuration dependencies across the project, replacing coq-core runtime references with rocq-runtime equivalents. The changes affect plugin library dependencies and package metadata declarations while maintaining functional structure. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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. Review rate limit: 0/1 reviews remaining, refill in 60 minutes.Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/plugin/META.coq-hammer (1)
3-3: Addrocq-runtime.vernacto META requires for consistency with dune dependencies, or document the intentional omission.The dune file lists
rocq-runtime.vernacas a library dependency (line 7), butsrc/plugin/META.coq-hammerrequires onlyrocq-runtime.plugins.ltac coq-hammer-tactics.lib. Ifrocq-runtime.vernacis transitively available throughrocq-runtime.plugins.ltac, this can remain as-is; otherwise, add it to the requires field to ensure runtime consistency with build-time dependencies.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/plugin/META.coq-hammer` at line 3, The META requires line currently lists "rocq-runtime.plugins.ltac coq-hammer-tactics.lib" but the dune file declares a dependency on rocq-runtime.vernac; update the META entry to include "rocq-runtime.vernac" in the requires field to match dune, or alternatively add a brief comment in META.coq-hammer stating that rocq-runtime.vernac is intentionally omitted because it is provided transitively via rocq-runtime.plugins.ltac; edit the requires field or add the explanatory comment accordingly to keep build/runtime dependencies consistent.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@src/plugin/META.coq-hammer`:
- Line 3: The META requires line currently lists "rocq-runtime.plugins.ltac
coq-hammer-tactics.lib" but the dune file declares a dependency on
rocq-runtime.vernac; update the META entry to include "rocq-runtime.vernac" in
the requires field to match dune, or alternatively add a brief comment in
META.coq-hammer stating that rocq-runtime.vernac is intentionally omitted
because it is provided transitively via rocq-runtime.plugins.ltac; edit the
requires field or add the explanatory comment accordingly to keep build/runtime
dependencies consistent.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 57a63a10-6d6b-46ba-97b9-b159ffa6bd99
📒 Files selected for processing (5)
src/lib/dunesrc/plugin/META.coq-hammersrc/plugin/dunesrc/tactics/META.coq-hammer-tacticssrc/tactics/dune
No description provided.